diff options
| author | cjihrig <cjihrig@gmail.com> | 2017-05-19 12:18:20 -0400 |
|---|---|---|
| committer | Myles Borins <mylesborins@google.com> | 2017-07-11 17:43:49 +0100 |
| commit | f92e065d120c03a8bfc67dbdc3be69755cbc2422 (patch) | |
| tree | 3a2b6c5a85f7ea1eea3604c11578d89e3395f62d /src/cares_wrap.cc | |
| parent | 18cbee236d48d72fc089720bbbf51c01e3749885 (diff) | |
| download | node-new-f92e065d120c03a8bfc67dbdc3be69755cbc2422.tar.gz | |
src: assert that uv_async_init() succeeds
This commit fixes two coverity warnings for unchecked
return values.
Refs: https://github.com/nodejs/node/pull/13050
PR-URL: https://github.com/nodejs/node/pull/13116
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Diffstat (limited to 'src/cares_wrap.cc')
| -rw-r--r-- | src/cares_wrap.cc | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/cares_wrap.cc b/src/cares_wrap.cc index 75cdc599a0..44d0926ba6 100644 --- a/src/cares_wrap.cc +++ b/src/cares_wrap.cc @@ -449,7 +449,9 @@ class QueryWrap : public AsyncWrap { data->len = answer_len; uv_async_t* async_handle = &data->async_handle; - uv_async_init(wrap->env()->event_loop(), async_handle, CaresAsyncCb); + CHECK_EQ(0, uv_async_init(wrap->env()->event_loop(), + async_handle, + CaresAsyncCb)); async_handle->data = data; uv_async_send(async_handle); @@ -472,7 +474,9 @@ class QueryWrap : public AsyncWrap { data->is_host = true; uv_async_t* async_handle = &data->async_handle; - uv_async_init(wrap->env()->event_loop(), async_handle, CaresAsyncCb); + CHECK_EQ(0, uv_async_init(wrap->env()->event_loop(), + async_handle, + CaresAsyncCb)); async_handle->data = data; uv_async_send(async_handle); |