From cad980c83fc2bb7d8634e6f7fcbe3cce4c2a31aa Mon Sep 17 00:00:00 2001 From: Peter Krempa Date: Thu, 4 Dec 2025 14:38:19 +0100 Subject: [PATCH] qemu: Use 'virQEMUSaveFormat' type everywhere except qemu_conf Convert all code refering to the save image type to use the proper enum value. Signed-off-by: Peter Krempa Reviewed-by: Michal Privoznik --- 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.47.3