[libvirt] [PATCH 1/3] security_dac: check if virSecurityDACGetIds returns negative

Ján Tomko jtomko at redhat.com
Fri Nov 20 10:00:14 UTC 2015


Use the customary check '< 0' instead of checking for non-zero.

No functional change.
---
 src/security/security_dac.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/security/security_dac.c b/src/security/security_dac.c
index dfdeffd..8b9c473 100644
--- a/src/security/security_dac.c
+++ b/src/security/security_dac.c
@@ -547,7 +547,7 @@ virSecurityDACSetSecurityHostdevLabelHelper(const char *file,
     uid_t user;
     gid_t group;
 
-    if (virSecurityDACGetIds(secdef, priv, &user, &group, NULL, NULL))
+    if (virSecurityDACGetIds(secdef, priv, &user, &group, NULL, NULL) < 0)
         return -1;
 
     return virSecurityDACSetOwnership(priv, NULL, file, user, group);
-- 
2.4.6




More information about the libvir-list mailing list