On Fri, Aug 21, 2020 at 16:34:30 +0200, Michal Privoznik wrote:
> Even if namespaces are disabled, then due to a missing check at the
> beginning of qemuDomainBuildNamespace(), the domain startup code
> still tries to populate (nonexistent) domain's namespace.
>
> Fixes: 8da362fe62766b4eee209cd3ce591ceb62299d13
> Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
> ---
> src/qemu/qemu_namespace.c | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/src/qemu/qemu_namespace.c b/src/qemu/qemu_namespace.c
> index 797722b714..b6724a095f 100644
> --- a/src/qemu/qemu_namespace.c
> +++ b/src/qemu/qemu_namespace.c
> @@ -616,6 +616,11 @@ qemuDomainBuildNamespace(virQEMUDriverConfigPtr cfg,
> {
> VIR_AUTOSTRINGLIST paths = NULL;
>
> + if (!qemuDomainNamespaceEnabled(vm, QEMU_DOMAIN_NS_MOUNT)) {
> + VIR_DEBUG("namespaces disabled for domain %s", vm->def->name);
> + return 0;
> + }
> +
> if (qemuDomainPopulateDevices(cfg, &paths) < 0)
> return -1;
Reviewed-by: Jiri Denemark <jdenemar@redhat.com>