]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
qemu: Fix crash in migration of graphics-less guests.
authorViktor Mihajlovski <mihajlov@linux.vnet.ibm.com>
Thu, 16 May 2013 12:41:29 +0000 (14:41 +0200)
committerJán Tomko <jtomko@redhat.com>
Fri, 31 May 2013 20:36:32 +0000 (22:36 +0200)
Commit 7f15ebc7a2b599ab10dbc15bca6f823591213e67 introduced a bug
happening when guests without a <graphics> element are migrated.
The initialization of listenAddress happens unconditionally
from the cookie even if the cookie->graphics pointer was NULL.
Moved the initialization to where it is safe.

Signed-off-by: Viktor Mihajlovski <mihajlov@linux.vnet.ibm.com>
(cherry picked from commit 9684bb11fd3832582308d0bcdb649041fd6584e2)

src/qemu/qemu_migration.c

index f602b2a07cbfcec19b006c2b77de786c3f2861a2..d96805a476bbffe0dac9851a7599124bff879afb 100644 (file)
@@ -1742,7 +1742,7 @@ qemuDomainMigrateGraphicsRelocate(virQEMUDriverPtr driver,
 {
     qemuDomainObjPrivatePtr priv = vm->privateData;
     int ret;
-    char *listenAddress = cookie->graphics->listen;
+    char *listenAddress;
 
     if (!cookie)
         return 0;
@@ -1756,6 +1756,7 @@ qemuDomainMigrateGraphicsRelocate(virQEMUDriverPtr driver,
     if (cookie->graphics->type != VIR_DOMAIN_GRAPHICS_TYPE_SPICE)
         return 0;
 
+    listenAddress = cookie->graphics->listen;
     if (!listenAddress ||
         STREQ(listenAddress, "0.0.0.0") ||
         STREQ(listenAddress, "::"))