diff options
author | Fedor Indutny <fedor@indutny.com> | 2016-06-03 14:52:38 -0400 |
---|---|---|
committer | Fedor Indutny <fedor@indutny.com> | 2016-06-03 14:52:38 -0400 |
commit | 201448c9bbd6a143e0ad04b4961bd856eb6b5012 (patch) | |
tree | faa3efdb7785c1cc6953d67214028db6e39d6dd0 /test/src/test-close.c | |
parent | 88ffbdbebace7624f934e91c97731f18f3f8ee03 (diff) | |
download | uv_link_t-201448c9bbd6a143e0ad04b4961bd856eb6b5012.tar.gz |
src: error or NULL links they happen on close
Right after `uv_link_close()` all links will be unchained, meaning that they `link->parent` is NULL. Instead of crashing on calls like: `uv_link_try_write(link->parent, ...)` Make them return `UV_EFAULT`, indicating that no parent is present anymore. Alternatively, these checks could be done in user code, but it will make it too cumbersome.
Diffstat (limited to 'test/src/test-close.c')
-rw-r--r-- | test/src/test-close.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/test/src/test-close.c b/test/src/test-close.c index bbbaf26..4f10630 100644 --- a/test/src/test-close.c +++ b/test/src/test-close.c @@ -19,10 +19,16 @@ static void c_close_cb(uv_link_t* l) { static void b_read_cb(uv_link_observer_t* o, ssize_t nread, const uv_buf_t* d) { b_reads |= 1 << nread; if (nread == 2) { + uv_buf_t tmp; + CHECK_EQ(a.close_depth, 0, "depth check"); CHECK_EQ(b.close_depth, 2, "depth check"); CHECK_EQ(c.close_depth, 1, "depth check"); uv_link_close((uv_link_t*) &c, c_close_cb); + + CHECK_EQ(o->parent, NULL, "parent should be zeroed"); + CHECK_EQ(uv_link_try_write(o->parent, &tmp, 0), UV_EFAULT, + "write should fail"); } else { CHECK_EQ(a.close_depth, 0, "depth check"); CHECK_EQ(b.close_depth, 1, "depth check"); |