Once upon a time dirname(cfg->channelTargetDir) was a unique dir, but nowadays it is the same as cfg->stateDir, so this is redundant. Signed-off-by: Cole Robinson <crobinso@redhat.com> --- src/qemu/qemu_driver.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 0cf88b8be9..5dff049d85 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -523,7 +523,6 @@ qemuStateInitializeDirs(bool privileged, virQEMUDriverConfig *cfg) { size_t i; - g_autofree char *channeldir = g_path_get_dirname(cfg->channelTargetDir); struct dirperms { const char *dir; @@ -540,7 +539,6 @@ qemuStateInitializeDirs(bool privileged, { cfg->slirpStateDir, 0777, cfg->user, cfg->group }, { cfg->passtStateDir, 0777, cfg->user, cfg->group }, { cfg->rdpStateDir, 0777, cfg->user, cfg->group }, - { channeldir, 0777, cfg->user, cfg->group }, { cfg->channelTargetDir, 0777, cfg->user, cfg->group }, /* example: /var/lib/libvirt/qemu */ -- 2.53.0