From 2a6825c393157fe476763565e94451bf78ccca8f Mon Sep 17 00:00:00 2001 From: "Daniel P. Berrange" Date: Wed, 26 Aug 2009 15:42:31 +0100 Subject: [PATCH] Automatically set correct ownership of QEMU state directories * src/qemu_driver.c: Change ownership of /var/{lib,cache}/libvirt/qemu to match user/group that QEMU VMs are configured to run as. --- src/qemu_driver.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/src/qemu_driver.c b/src/qemu_driver.c index 5ad434f4c3..d424ce7bb0 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -572,6 +572,21 @@ qemudStartup(int privileged) { goto error; } + if (privileged) { + if (chown(qemu_driver->libDir, qemu_driver->user, qemu_driver->group) < 0) { + virReportSystemError(NULL, errno, + _("unable to set ownership of '%s' to user %d:%d"), + qemu_driver->libDir, qemu_driver->user, qemu_driver->group); + goto error; + } + if (chown(qemu_driver->cacheDir, qemu_driver->user, qemu_driver->group) < 0) { + virReportSystemError(NULL, errno, + _("unable to set ownership of '%s' to %d:%d"), + qemu_driver->cacheDir, qemu_driver->user, qemu_driver->group); + goto error; + } + } + if (qemudSecurityInit(qemu_driver) < 0) { goto error; }