[PATCH 4/9] qemu: Use 'virQEMUSaveFormat' type everywhere except qemu_conf

Peter Krempa via Devel posted 9 patches 3 days, 3 hours ago
[PATCH 4/9] qemu: Use 'virQEMUSaveFormat' type everywhere except qemu_conf
Posted by Peter Krempa via Devel 3 days, 3 hours ago
From: Peter Krempa <pkrempa@redhat.com>

Convert all code refering to the save image type to use the proper enum
value.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
---
 src/qemu/qemu_driver.c    | 18 ++++++++++++------
 src/qemu/qemu_saveimage.c |  4 ++--
 src/qemu/qemu_saveimage.h |  4 ++--
 3 files changed, 16 insertions(+), 10 deletions(-)

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index f2e024dae3..d88c0833ad 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -2583,7 +2583,7 @@ static int
 qemuDomainSaveInternal(virQEMUDriver *driver,
                        virDomainObj *vm,
                        const char *path,
-                       int format,
+                       virQEMUSaveFormat format,
                        virCommand *compressor,
                        const char *xmlin,
                        virTypedParameterPtr params,
@@ -2823,7 +2823,7 @@ qemuDomainSaveParams(virDomainPtr dom,
     const char *to = NULL;
     const char *dxml = NULL;
     const char *formatstr = NULL;
-    int format = cfg->saveImageFormat;
+    virQEMUSaveFormat format = cfg->saveImageFormat;
     int ret = -1;

     virCheckFlags(VIR_DOMAIN_SAVE_BYPASS_CACHE |
@@ -2863,10 +2863,16 @@ qemuDomainSaveParams(virDomainPtr dom,
         return qemuDomainManagedSaveHelper(driver, vm, dxml, flags);
     }

-    if (formatstr && (format = qemuSaveFormatTypeFromString(formatstr)) < 0) {
-        virReportError(VIR_ERR_OPERATION_FAILED,
-                       _("Invalid image_format '%1$s'"), formatstr);
-        goto cleanup;
+    if (formatstr) {
+        int formatVal;
+
+        if ((formatVal = qemuSaveFormatTypeFromString(formatstr)) < 0) {
+            virReportError(VIR_ERR_OPERATION_FAILED,
+                           _("Invalid image_format '%1$s'"), formatstr);
+            goto cleanup;
+        }
+
+        format = formatVal;
     }

     if (qemuSaveImageGetCompressionProgram(format, &compressor, "save") < 0)
diff --git a/src/qemu/qemu_saveimage.c b/src/qemu/qemu_saveimage.c
index 145a0f4832..48f8220dee 100644
--- a/src/qemu/qemu_saveimage.c
+++ b/src/qemu/qemu_saveimage.c
@@ -79,7 +79,7 @@ virQEMUSaveData *
 virQEMUSaveDataNew(char *domXML,
                    qemuDomainSaveCookie *cookieObj,
                    bool running,
-                   int format,
+                   virQEMUSaveFormat format,
                    virDomainXMLOption *xmlopt)
 {
     virQEMUSaveData *data = NULL;
@@ -551,7 +551,7 @@ qemuSaveImageCreate(virQEMUDriver *driver,
  * Returns -1 on failure, 0 on success.
  */
 int
-qemuSaveImageGetCompressionProgram(int format,
+qemuSaveImageGetCompressionProgram(virQEMUSaveFormat format,
                                    virCommand **compressor,
                                    const char *styleFormat)
 {
diff --git a/src/qemu/qemu_saveimage.h b/src/qemu/qemu_saveimage.h
index 15b73eb395..0a22ee5f05 100644
--- a/src/qemu/qemu_saveimage.h
+++ b/src/qemu/qemu_saveimage.h
@@ -114,7 +114,7 @@ qemuSaveImageOpen(virQEMUDriver *driver,
     ATTRIBUTE_NONNULL(2) ATTRIBUTE_NONNULL(4);

 int
-qemuSaveImageGetCompressionProgram(int format,
+qemuSaveImageGetCompressionProgram(virQEMUSaveFormat format,
                                    virCommand **compressor,
                                    const char *styleFormat)
     ATTRIBUTE_NONNULL(2);
@@ -153,7 +153,7 @@ virQEMUSaveData *
 virQEMUSaveDataNew(char *domXML,
                    qemuDomainSaveCookie *cookieObj,
                    bool running,
-                   int format,
+                   virQEMUSaveFormat format,
                    virDomainXMLOption *xmlopt);

 void
-- 
2.52.0