]> xenbits.xensource.com Git - libvirt.git/commitdiff
virSecuritySELinuxSetFileconHelper: Fix build with broken selinux.h
authorMichal Privoznik <mprivozn@redhat.com>
Tue, 10 Jan 2017 15:33:58 +0000 (16:33 +0100)
committerAndrea Bolognani <abologna@redhat.com>
Tue, 10 Jan 2017 18:23:49 +0000 (19:23 +0100)
There are still some systems out there that have broken
setfilecon*() prototypes. Instead of taking 'const char *tcon' it
is taking 'char *tcon'. The function should just set the context,
not modify it.

We had been bitten with this problem before which resulted in
292d3f2d and subsequently b109c09765. However, with one my latest
commits (4674fc6afd6d) I've changed the type of @tcon variable to
'const char *' which results in build failure on the systems from
above.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
src/security/security_selinux.c

index 92267e8fc81a3ba8998604d2cd4b3a4044344d8e..f229b5139ac0c7f69b2486bc8d3535eca8063899 100644 (file)
@@ -1138,7 +1138,7 @@ virSecuritySELinuxSetFileconHelper(const char *path, const char *tcon,
 
     VIR_INFO("Setting SELinux context on '%s' to '%s'", path, tcon);
 
-    if (setfilecon_raw(path, tcon) < 0) {
+    if (setfilecon_raw(path, (VIR_SELINUX_CTX_CONST char *) tcon) < 0) {
         int setfilecon_errno = errno;
 
         if (getfilecon_raw(path, &econ) >= 0) {