From: Kamalesh Babulal <kamalesh.babulal@oracle.com>
Date: Tue, 25 Oct 2022 21:28:51 +0000 (-0600)
Subject: api.c: fix resource leak in cg_chmod_path()
X-Git-Tag: v2.0.3~7
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=38f026747e433ba125eba8baa7940fb6a0f08256;p=thirdparty%2Flibcgroup.git

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 <kamalesh.babulal@oracle.com>
Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
---

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;