[libvirt PATCH 11/23] src: remove use of the term 'whitelist' from remaining code
Ján Tomko
jtomko at redhat.com
Tue Jun 23 08:20:29 UTC 2020
On a Monday in 2020, Daniel P. Berrangé wrote:
>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.
>
I am specifically talking about the context. I wrote the comment to
contrast against the new-style sandbox.
If you don't want to preserve the contrast in here, just delete the
comment.
Jano
>
>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 :|
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20200623/cc14a252/attachment-0001.sig>
More information about the libvir-list
mailing list