[libvirt] [PATCH] Fixed connection definition for non-SELinux builds

[Eric Blake]

On 02/03/2012 08:03 AM, Martin Kletzander wrote:
> This patch fixes the access of variable "con" in two files where the
> variable was declared only on SELinux builds and thus the build failed
> without SELinux. It's a rather nasty fix but helps fix the build
> quickly and without any major changes to the code.
> ---
>  src/lxc/lxc_container.c  |    2 ++
>  src/lxc/lxc_controller.c |    2 ++
>  2 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/src/lxc/lxc_container.c b/src/lxc/lxc_container.c
> index d1156e4..1c148e2 100644
> --- a/src/lxc/lxc_container.c
> +++ b/src/lxc/lxc_container.c
> @@ -449,6 +449,8 @@ static int lxcContainerMountBasicFS(const char *srcprefix, bool pivotRoot)
>      char *opts = NULL;
>  #if HAVE_SELINUX
>      security_context_t con;
> +#else
> +    bool con = false;

I would have used 'char *con = NULL', as security_context_t is more
likely to be a pointer type than a boolean, and since we already have a
'(const char*)con' cast at one point in the code (casting a bool to
char* is fishy).  But I guess your hack is okay.

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120203/4c4173b2/attachment-0001.sig>