[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCHv1 2/4] qemu: conf: add sandbox option



On Tue, Sep 11, 2012 at 04:52:48PM +0200, Ján Tomko wrote:
> On 09/11/12 15:11, Daniel P. Berrange wrote:
> > On Tue, Sep 11, 2012 at 02:58:19PM +0200, Ján Tomko wrote:
> >> diff --git a/src/qemu/qemu_conf.h b/src/qemu/qemu_conf.h
> >> index ac285f6..34b952f 100644
> >> --- a/src/qemu/qemu_conf.h
> >> +++ b/src/qemu/qemu_conf.h
> >> @@ -152,6 +152,7 @@ struct qemud_driver {
> >>  
> >>      int keepAliveInterval;
> >>      unsigned int keepAliveCount;
> >> +    int sandbox;
> > 
> > s/int/bool/
> 
> bool won't allow three states - on, off and QEMU default. I think the
> third option could be useful. The default in QEMU 1.2 is off, with a
> plan to turn it on by default in 1.3.
> 
> https://www.redhat.com/archives/libvir-list/2012-September/msg00415.html

Oh true, ignore me.  Just rename this var to 'seccompSandbox'

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]