cgrulesengd: Fix Coverity warnings about a socket leak

This commit resolves the following Coverity warnings:

CID 1412123 (#1 of 1): Resource leak (RESOURCE_LEAK)12. leaked_handle:
Handle variable sk_nl going out of scope leaks the handle.

CID 1412129 (#1 of 1): Resource leak (RESOURCE_LEAK)13. leaked_handle:
Handle variable sk_unix going out of scope leaks the handle.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

diff --git a/src/daemon/cgrulesengd.c b/src/daemon/cgrulesengd.c
index 5efc00eb9aed6d4745cb953b6f9b261bb8d8a7ea..4cef53ee0e10bdd235bd3f8fcfe2b011b0e479de 100644 (file)
--- a/src/daemon/cgrulesengd.c
+++ b/src/daemon/cgrulesengd.c
@@ -792,9 +792,9 @@ static int cgre_create_netlink_socket_process_msg(void)
        }
 
 close_and_exit:
-       if (sk_nl > 0)
+       if (sk_nl >= 0)
                close(sk_nl);
-       if (sk_unix > 0)
+       if (sk_unix >= 0)
                close(sk_unix);
        return rc;
 }