[libvirt] [PATCH 2/2] build: avoid compiler warning on cygwin

Eric Blake eblake at redhat.com
Wed Mar 9 04:58:54 UTC 2011


On cygwin:

  CC       libvirt_driver_security_la-security_dac.lo
security/security_dac.c: In function 'virSecurityDACSetProcessLabel':
security/security_dac.c:618: warning: format '%d' expects type 'int', but argument 7 has type 'uid_t' [-Wformat]

We've done this before (see src/util/util.c).

* src/security/security_dac.c (virSecurityDACSetProcessLabel): On
cygwin, uid_t is a 32-bit long.
---

Pushing under the trivial rule (I can't yet enable -Werror on cygwin,
because the <rpc.h> headers from tirpc aren't compile-clean - they
redeclare setrpcent - otherwise I would be pushing under the
build-breaker rule).

 src/security/security_dac.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/src/security/security_dac.c b/src/security/security_dac.c
index b8de232..fba2d1d 100644
--- a/src/security/security_dac.c
+++ b/src/security/security_dac.c
@@ -615,7 +615,8 @@ virSecurityDACSetProcessLabel(virSecurityManagerPtr mgr,
 {
     virSecurityDACDataPtr priv = virSecurityManagerGetPrivateData(mgr);

-    VIR_DEBUG("Dropping privileges of VM to %d:%d", priv->user, priv->group);
+    VIR_DEBUG("Dropping privileges of VM to %u:%u",
+              (unsigned int) priv->user, (unsigned int) priv->group);

     if (virSetUIDGID(priv->user, priv->group) < 0)
         return -1;
-- 
1.7.4




More information about the libvir-list mailing list