The only thing created directly in channelTargetDir is domain specific directories (see priv->channelTargetDir). 0755 seems fine here, like other state dirs. Add channelTargetDir to the rpm spec similarly Signed-off-by: Cole Robinson <crobinso@redhat.com> --- libvirt.spec.in | 1 + src/qemu/qemu_driver.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 258c59e7c5..e001a1ac64 100644 --- a/libvirt.spec.in +++ b/libvirt.spec.in @@ -2338,6 +2338,7 @@ exit 0 %config(noreplace) %{_sysconfdir}/libvirt/qemu-lockd.conf %config(noreplace) %{_sysconfdir}/logrotate.d/libvirtd.qemu %ghost %dir %attr(0755, %{qemu_user}, %{qemu_group}) %{_rundir}/libvirt/qemu/ +%ghost %dir %attr(0755, %{qemu_user}, %{qemu_group}) %{_rundir}/libvirt/qemu/channel/ %ghost %dir %attr(0770, %{qemu_user}, %{qemu_group}) %{_rundir}/libvirt/qemu/dbus/ %ghost %dir %attr(0755, %{qemu_user}, %{qemu_group}) %{_rundir}/libvirt/qemu/passt/ %ghost %dir %attr(0755, %{qemu_user}, %{qemu_group}) %{_rundir}/libvirt/qemu/rdp/ diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index cf4f97d104..99b752f7e8 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -542,7 +542,7 @@ qemuStateInitializeDirs(bool privileged, { cfg->slirpStateDir, 0755, cfg->user, cfg->group }, { cfg->passtStateDir, 0755, cfg->user, cfg->group }, { cfg->rdpStateDir, 0755, cfg->user, cfg->group }, - { cfg->channelTargetDir, 0777, cfg->user, cfg->group }, + { cfg->channelTargetDir, 0755, cfg->user, cfg->group }, /* example: /var/lib/libvirt/qemu */ { cfg->libDir, 0751, cfg->user, cfg->group }, -- 2.53.0