From: Martin Kletzander <mkletzan@redhat.com>
Date: Thu, 13 Aug 2015 05:35:10 +0000 (+0200)
Subject: security_selinux: Use proper structure to access socket data
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=4ac6ce38d3bc60f45491509ede4fc8437348048c;p=people%2Fliuw%2Flibxenctrl-split%2Flibvirt.git

security_selinux: Use proper structure to access socket data

In virSecuritySELinuxSetSecurityChardevLabel() we are labelling unix
socket path, but accessing another structure of the union.  This does
not pose a problem currently as both paths are at the same offset, but
this should be fixed for the future.

Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
---

diff --git a/src/security/security_selinux.c b/src/security/security_selinux.c
index 6e67a86cc..a37f8c140 100644
--- a/src/security/security_selinux.c
+++ b/src/security/security_selinux.c
@@ -1740,7 +1740,7 @@ virSecuritySELinuxSetSecurityChardevLabel(virDomainDefPtr def,
 
     case VIR_DOMAIN_CHR_TYPE_UNIX:
         if (!dev_source->data.nix.listen) {
-            if (virSecuritySELinuxSetFilecon(dev_source->data.file.path,
+            if (virSecuritySELinuxSetFilecon(dev_source->data.nix.path,
                                              imagelabel) < 0)
                 goto done;
         }