Commit c03bd57
src: check uv_async_init() return value
Pointed out by Coverity.
PR-URL: #7374
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>1 parent 1baa145 commit c03bd57
2 files changed
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -3857,9 +3857,9 @@ void Init(int* argc, | |||
| 3857 | 3857 | ||
| 3858 | 3858 | // init async debug messages dispatching | |
| 3859 | 3859 | // Main thread uses uv_default_loop | |
| 3860 | - uv_async_init(uv_default_loop(), | ||
| 3861 | - &dispatch_debug_messages_async, | ||
| 3862 | - DispatchDebugMessagesAsyncCallback); | ||
| 3860 | + CHECK_EQ(0, uv_async_init(uv_default_loop(), | ||
| 3861 | + &dispatch_debug_messages_async, | ||
| 3862 | + DispatchDebugMessagesAsyncCallback)); | ||
| 3863 | 3863 | uv_unref(reinterpret_cast<uv_handle_t*>(&dispatch_debug_messages_async)); | |
| 3864 | 3864 | ||
| 3865 | 3865 | #if defined(NODE_V8_OPTIONS) | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -155,9 +155,9 @@ void init_etw() { | |||
| 155 | 155 | event_write = (EventWriteFunc)GetProcAddress(advapi, "EventWrite"); | |
| 156 | 156 | ||
| 157 | 157 | // create async object used to invoke main thread from callback | |
| 158 | - uv_async_init(uv_default_loop(), | ||
| 159 | - &dispatch_etw_events_change_async, | ||
| 160 | - etw_events_change_async); | ||
| 158 | + CHECK_EQ(0, uv_async_init(uv_default_loop(), | ||
| 159 | + &dispatch_etw_events_change_async, | ||
| 160 | + etw_events_change_async)); | ||
| 161 | 161 | uv_unref(reinterpret_cast<uv_handle_t*>(&dispatch_etw_events_change_async)); | |
| 162 | 162 | ||
| 163 | 163 | if (event_register) { | |
0 commit comments