[libvirt] [PATCH] util: Properly return error from virGetUserID and virGetGroupID stubs

Peter Krempa pkrempa at redhat.com
Mon Jun 8 07:35:44 UTC 2015


The stubs for the two functions that are compiled on platforms that
don't have HAVE_GETPWUID_R and friends defined do not return error but
report an error message. The calling code then assumes that the @uid or
@gid arguments were filled, which is not the case in the stubs.
---

This should fix the mingw build that complains that @theuid in one of the
callers is not initialized.

 src/util/virutil.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/util/virutil.c b/src/util/virutil.c
index e479cce..cddc78a 100644
--- a/src/util/virutil.c
+++ b/src/util/virutil.c
@@ -1316,7 +1316,7 @@ int virGetUserID(const char *name ATTRIBUTE_UNUSED,
     virReportError(VIR_ERR_INTERNAL_ERROR,
                    "%s", _("virGetUserID is not available"));

-    return 0;
+    return -1;
 }


@@ -1326,7 +1326,7 @@ int virGetGroupID(const char *name ATTRIBUTE_UNUSED,
     virReportError(VIR_ERR_INTERNAL_ERROR,
                    "%s", _("virGetGroupID is not available"));

-    return 0;
+    return -1;
 }

 int
-- 
2.4.1




More information about the libvir-list mailing list