qemu: Reset the node-name allocator in qemuDomainObjPrivateDataClear

qemuDomainObjPrivateDataClear clears state which become invalid after VM
stopped running and the node name allocator belongs there.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>

diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index f545d0b8885cd1c5fde28a84232acc5fc60314a3..ee9562b9d9b545fa0ae84e7469f69cdd367a1c02 100644 (file)
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -2236,6 +2236,9 @@ qemuDomainObjPrivateDataClear(qemuDomainObjPrivatePtr priv)
 
     virDomainBackupDefFree(priv->backup);
     priv->backup = NULL;
+
+    /* reset node name allocator */
+    qemuDomainStorageIdReset(priv);
 }
 
 

diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index 114229e373aed4bb01aa7aa9122a9157ab233d63..0a886378cd972e7cfb061bac07e9898372a8e7fd 100644 (file)
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -7567,9 +7567,6 @@ void qemuProcessStop(virQEMUDriverPtr driver,
     /* clear all private data entries which are no longer needed */
     qemuDomainObjPrivateDataClear(priv);
 
-    /* reset node name allocator */
-    qemuDomainStorageIdReset(priv);
-
     /* The "release" hook cleans up additional resources */
     if (virHookPresent(VIR_HOOK_DRIVER_QEMU)) {
         g_autofree char *xml = qemuDomainDefFormatXML(driver, NULL, vm->def, 0);