[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