From ef728ceea316503bdfd75c386512045fc8aa8285 Mon Sep 17 00:00:00 2001 From: Mathieu Desnoyers Date: Thu, 19 Sep 2019 10:10:31 -0400 Subject: [PATCH] Fix: provide errno as argument to urcu_die() commit 1a990de3add "Fix: rculfhash worker needs to unblock to SIGRCU" provides "ret" (-1) as argument to urcu_die(), but should rather provide errno. Reported by Coverity: ** CID 1405700: Error handling issues (NEGATIVE_RETURNS) /src/rculfhash.c: 2171 in cds_lfht_worker_init() Signed-off-by: Mathieu Desnoyers --- src/rculfhash.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/rculfhash.c b/src/rculfhash.c index 17d3143..5950626 100644 --- a/src/rculfhash.c +++ b/src/rculfhash.c @@ -2123,7 +2123,7 @@ static void cds_lfht_worker_init(struct urcu_workqueue *workqueue, urcu_die(errno); ret = sigdelset(&mask, SIGRCU); if (ret) - urcu_die(ret); + urcu_die(errno); ret = pthread_sigmask(SIG_SETMASK, &mask, NULL); if (ret) urcu_die(ret); -- 2.34.1