From 38f026747e433ba125eba8baa7940fb6a0f08256 Mon Sep 17 00:00:00 2001 From: Kamalesh Babulal Date: Tue, 25 Oct 2022 15:28:51 -0600 Subject: [PATCH] api.c: fix resource leak in cg_chmod_path() Fix resource leak, reported by Coverity Tool: CID 276160 (#1 of 1): Resource leak (RESOURCE_LEAK)10. leaked_handle: Handle variable fd going out of scope leaks the handle. In cg_chmod_path(), the file descriptor checks for fd > 0, Coverity warns about the possibility of the file descriptor being 0, fix it by changing the check from '0' -> '-1'. Fixes: d569122a8c5f ("api.c: fix file open in cg_chmod_path()") Signed-off-by: Kamalesh Babulal Signed-off-by: Tom Hromatka --- src/api.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/api.c b/src/api.c index 8b7c665f..2c30ad32 100644 --- a/src/api.c +++ b/src/api.c @@ -240,7 +240,7 @@ fail: strerror(errno)); last_errno = errno; - if (fd > 0) + if (fd > -1) close(fd); return ECGOTHER; -- 2.47.2