[libvirt PATCH 11/23] src: remove use of the term 'whitelist' from remaining code

Daniel P. Berrangé berrange at redhat.com
Mon Jun 22 16:43:16 UTC 2020


On Fri, Jun 19, 2020 at 01:43:46PM +0200, Ján Tomko wrote:
> On a Friday in 2020, Daniel P. Berrangé wrote:
> > The terms can be avoided with simple tweaks.
> > 
> > Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
> > ---
> > src/libvirt.c           | 2 +-
> > src/qemu/qemu_command.c | 2 +-
> > src/qemu/qemu_conf.c    | 4 ++--
> > src/rpc/gendispatch.pl  | 2 +-
> > 4 files changed, 5 insertions(+), 5 deletions(-)
> > 
> > diff --git a/src/libvirt.c b/src/libvirt.c
> > index b2d0ba3d23..a0a21fd5d2 100644
> > --- a/src/libvirt.c
> > +++ b/src/libvirt.c
> > @@ -1025,7 +1025,7 @@ virConnectOpenInternal(const char *name,
> >             bool matchScheme = false;
> >             size_t s;
> >             if (!ret->uri) {
> > -                VIR_DEBUG("No URI, skipping driver with URI whitelist");
> > +                VIR_DEBUG("No URI, skipping driver with URI scheme filtering");
> >                 continue;
> >             }
> >             if (embed && !virConnectDriverTab[i]->embeddable) {
> > diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
> > index 37113a433a..ff539b1556 100644
> > --- a/src/qemu/qemu_command.c
> > +++ b/src/qemu/qemu_command.c
> > @@ -9526,7 +9526,7 @@ qemuBuildSeccompSandboxCommandLine(virCommandPtr cmd,
> >         return 0;
> >     }
> > 
> > -    /* Seccomp whitelist is opt-in */
> > +    /* Seccomp sandbox is opt-in */
> 
> This is incorrect, we use the seccomp sandbox by default on newer QEMUs.

This comment is against the code that deals with the old style QEMU
seccomp code, which rquires an explicit opt-in. The new style seccom
is further up above the diff hunk context seen here.


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




More information about the libvir-list mailing list