From afae41aa40a5d71b7863eaccc06faaffed00c217 Mon Sep 17 00:00:00 2001 From: Tony Finch Date: Mon, 15 May 2023 15:35:57 +0100 Subject: [PATCH] Check the return value from uv_async_send() An omission pointed out by the following report from Coverity: /lib/isc/loop.c: 483 in isc_loopmgr_pause() >>> CID 455002: Error handling issues (CHECKED_RETURN) >>> Calling "uv_async_send" without checking return value (as is done elsewhere 5 out of 6 times). 483 uv_async_send(&loop->pause_trigger); --- lib/isc/loop.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/isc/loop.c b/lib/isc/loop.c index 0b3918a00f..ddbc83d37b 100644 --- a/lib/isc/loop.c +++ b/lib/isc/loop.c @@ -480,7 +480,8 @@ isc_loopmgr_pause(isc_loopmgr_t *loopmgr) { continue; } - uv_async_send(&loop->pause_trigger); + int r = uv_async_send(&loop->pause_trigger); + UV_RUNTIME_CHECK(uv_async_send, r); } RUNTIME_CHECK(atomic_compare_exchange_strong(&loopmgr->paused,