[libvirt] [PATCH v2 15/19] qemu: Reload virUdevMgr on start
Michal Privoznik
mprivozn at redhat.com
Thu Nov 3 12:19:05 UTC 2016
If the daemon is restarted, the virUdevMgr loses its internal
state. This is because entries to its internal DB are added
whilst setting security labels. This obviously doesn't happen
when the daemon is restarted.
It's not wise to start with a fresh internal state and possibly
leave behind previous one.
Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
---
src/qemu/qemu_driver.c | 18 +++++++++++++++---
1 file changed, 15 insertions(+), 3 deletions(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index d785c60..7848a4e 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -377,10 +377,20 @@ qemuSecurityInit(virQEMUDriverPtr driver)
virSecurityManagerPtr stack = NULL;
virQEMUDriverConfigPtr cfg = virQEMUDriverGetConfig(driver);
unsigned int flags = 0;
+ char *file = NULL;
- if (cfg->writeUdev &&
- !(driver->udevMgr = virUdevMgrNew()))
- goto error;
+ if (cfg->writeUdev) {
+ if (!(file = qemuProcessGetUdevPath(driver)))
+ goto error;
+
+ if (virFileExists(file)) {
+ if (!(driver->udevMgr = virUdevMgrNewFromFile(file)))
+ goto error;
+ } else {
+ if (!(driver->udevMgr = virUdevMgrNew()))
+ goto error;
+ }
+ }
if (cfg->allowDiskFormatProbing)
flags |= VIR_SECURITY_MANAGER_ALLOW_DISK_PROBE;
@@ -442,12 +452,14 @@ qemuSecurityInit(virQEMUDriverPtr driver)
}
driver->securityManager = stack;
+ VIR_FREE(file);
virObjectUnref(cfg);
return 0;
error:
virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
_("Failed to initialize security drivers"));
+ VIR_FREE(file);
virObjectUnref(stack);
virObjectUnref(mgr);
virObjectUnref(cfg);
--
2.8.4
More information about the libvir-list
mailing list