From: Peter Krempa <pkrempa@redhat.com>
Date: Mon, 7 Mar 2016 09:06:19 +0000 (+0100)
Subject: qemu: rename: Forbid renaming domains with managed save image
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=47fdf9bc3a823e21baa6fc8c504dfc3e97c560d9;p=libvirt.git

qemu: rename: Forbid renaming domains with managed save image

The code does not handle renaming of the save state file. In addition to
that the resuming code would need to be tweaked to handle the name
change since the XML is extracted from the save image. The easies option
is to make the rename API even less useful by forbiding this.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1314594
---

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 102fade280..4020364000 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -19994,6 +19994,12 @@ static int qemuDomainRename(virDomainPtr dom,
         goto endjob;
     }
 
+    if (vm->hasManagedSave) {
+        virReportError(VIR_ERR_OPERATION_INVALID, "%s",
+                       _("domain with a managed saved state can't be renamed"));
+        goto endjob;
+    }
+
     if (virDomainObjGetState(vm, NULL) != VIR_DOMAIN_SHUTOFF) {
         virReportError(VIR_ERR_OPERATION_INVALID,
                        "%s", _("domain has to be shutoff before renaming"));