test driver: Fix domain ID after redefining a running VM

The ID of the existing VM was being unconditionally set to -1, which was
upsetting virt-manager.

diff --git a/ChangeLog b/ChangeLog
index 4c77baa1b944f2590cf5e049c30d6642e7a35128..aeeb1bac1f12ddab3bc902f1ebf6479bbc9c7c0b 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+Mon Jun 22 11:09:18 EDT 2009 Cole Robinson <crobinso@redhat.com>
+
+       * src/test.c: Fix domain ID after redefining a running VM
+
 Mon Jun 22 12:27:39 CEST 2009 Daniel Veillard <veillard@redhat.com>
 
        * src/conf.[ch]: restrict VMX syntax parsing types, patch by Matthias

diff --git a/src/test.c b/src/test.c
index 7dc0840e6132cc24be9a2b7a6fa9044ae3bc9032..2a672a3f69f5c639271532c114910fd3dc49ff4c 100644 (file)
--- a/src/test.c
+++ b/src/test.c
@@ -1623,16 +1623,16 @@ static virDomainPtr testDomainDefineXML(virConnectPtr conn,
                                   def)) == NULL) {
         goto cleanup;
     }
+    def = NULL;
     dom->persistent = 1;
-    dom->def->id = -1;
+
     event = virDomainEventNewFromObj(dom,
                                      VIR_DOMAIN_EVENT_DEFINED,
                                      VIR_DOMAIN_EVENT_DEFINED_ADDED);
 
-    ret = virGetDomain(conn, def->name, def->uuid);
-    def = NULL;
+    ret = virGetDomain(conn, dom->def->name, dom->def->uuid);
     if (ret)
-        ret->id = -1;
+        ret->id = dom->def->id;
 
 cleanup:
     virDomainDefFree(def);