Re: [libvirt] [PATCH 2/6] Don't enable the AppArmour security driver with LXC

2012-05-14 Thread Daniel P. Berrange
On Mon, May 14, 2012 at 08:06:03AM -0500, Jamie Strandboge wrote: > On Fri, 2012-05-11 at 22:39 -0400, Stefan Berger wrote: > > > +if (virtDriver&& STREQ(virtDriver, "LXC")) > > > +return rc; > > > + > > ACK once changed to: > if (virtDriver && STREQ(virtDriver, "LXC")) The patch is

Re: [libvirt] [PATCH 2/6] Don't enable the AppArmour security driver with LXC

2012-05-14 Thread Jamie Strandboge
On Fri, 2012-05-11 at 22:39 -0400, Stefan Berger wrote: > > +if (virtDriver&& STREQ(virtDriver, "LXC")) > > +return rc; > > + ACK once changed to: if (virtDriver && STREQ(virtDriver, "LXC")) ... -- Jamie Strandboge | http://www.canonical.com signature.asc Description:

Re: [libvirt] [PATCH 2/6] Don't enable the AppArmour security driver with LXC

2012-05-11 Thread Stefan Berger
On 05/11/2012 06:10 AM, Daniel P. Berrange wrote: From: Daniel Walsh The AppArmour driver does not currently have support for LXC so ensure that when probing, it claims to be disabled Signed-off-by: Daniel P. Berrange --- src/security/security_apparmor.c |5 - 1 file changed, 4 inser

[libvirt] [PATCH 2/6] Don't enable the AppArmour security driver with LXC

2012-05-11 Thread Daniel P. Berrange
From: Daniel Walsh The AppArmour driver does not currently have support for LXC so ensure that when probing, it claims to be disabled Signed-off-by: Daniel P. Berrange --- src/security/security_apparmor.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/security/s