From: Peter Krempa <pkrempa@redhat.com>
Date: Wed, 25 Mar 2015 07:52:26 +0000 (+0100)
Subject: daemon: Clear fake domain def object that is used to check ACL prior to use
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=6ca857c7c8a1f7b571132d6c7fff5a06301a5e9a;p=libvirt.git

daemon: Clear fake domain def object that is used to check ACL prior to use

The fake object is used to pass the domain name and UUID to the ACL code
for events where we don't have the full domain def when dispatching
events. The rest of the entries would be left uninitialized. While this
is not a problem code-wise as the used fields are initialized it looks
ugly in the debugger.
---

diff --git a/daemon/remote.c b/daemon/remote.c
index ea7ae9485d..9e28720937 100644
--- a/daemon/remote.c
+++ b/daemon/remote.c
@@ -151,6 +151,7 @@ remoteRelayDomainEventCheckACL(virNetServerClientPtr client,
     /* For now, we just create a virDomainDef with enough contents to
      * satisfy what viraccessdriverpolkit.c references.  This is a bit
      * fragile, but I don't know of anything better.  */
+    memset(&def, 0, sizeof(def));
     def.name = dom->name;
     memcpy(def.uuid, dom->uuid, VIR_UUID_BUFLEN);