From 17d5edcd595c0f7dcd1833fe40507653fdd8a37b Mon Sep 17 00:00:00 2001 From: cjihrig Date: Fri, 19 May 2017 12:18:20 -0400 Subject: [PATCH] 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 Reviewed-By: Refael Ackermann Reviewed-By: Anna Henningsen Reviewed-By: Michael Dawson Reviewed-By: Gibson Fahnestock --- src/cares_wrap.cc | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/cares_wrap.cc b/src/cares_wrap.cc index 75cdc599a03a60..44d0926ba6b178 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);