From 98644869669274b6d7858a2327076c38ae9dcb9f Mon Sep 17 00:00:00 2001 From: Peter Krempa Date: Wed, 6 Dec 2023 14:32:09 +0100 Subject: [PATCH] qemuMigrationDstPrecreateStorage: Refactor cleanup MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Use automatic pointer freeing for 'conn' and remove the 'cleanup' label. Signed-off-by: Peter Krempa Reviewed-by: Ján Tomko --- src/qemu/qemu_migration.c | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c index b303b540cf..356b7c382b 100644 --- a/src/qemu/qemu_migration.c +++ b/src/qemu/qemu_migration.c @@ -440,9 +440,8 @@ qemuMigrationDstPrecreateStorage(virDomainObj *vm, const char **migrate_disks, bool incremental) { - int ret = -1; + g_autoptr(virConnect) conn = NULL; size_t i = 0; - virConnectPtr conn = NULL; if (!nbd || !nbd->ndisks) return 0; @@ -459,7 +458,7 @@ qemuMigrationDstPrecreateStorage(virDomainObj *vm, virReportError(VIR_ERR_INTERNAL_ERROR, _("unable to find disk target '%1$s' for non-shared-storage migration"), nbd->disks[i].target); - goto cleanup; + return -1; } if (disk->src->type == VIR_STORAGE_TYPE_NVME) { @@ -479,20 +478,16 @@ qemuMigrationDstPrecreateStorage(virDomainObj *vm, virReportError(VIR_ERR_OPERATION_UNSUPPORTED, _("pre-creation of storage target '%1$s' for incremental storage migration of disk '%2$s' is not supported"), NULLSTR(diskSrcPath), nbd->disks[i].target); - goto cleanup; + return -1; } VIR_DEBUG("Proceeding with disk source %s", NULLSTR(diskSrcPath)); - if (qemuMigrationDstPrecreateDisk(&conn, - disk, nbd->disks[i].capacity) < 0) - goto cleanup; + if (qemuMigrationDstPrecreateDisk(&conn, disk, nbd->disks[i].capacity) < 0) + return -1; } - ret = 0; - cleanup: - virObjectUnref(conn); - return ret; + return 0; }