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 <[email protected]>
---
 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

Reply via email to