From: Kirill Shchetiniuk <kshcheti@redhat.com> Moved variables to closer scope to enable suto memory management Signed-off-by: Kirill Shchetiniuk <kshcheti@redhat.com> --- src/qemu/qemu_driver.c | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index c480bb59e1..eb4e9c1c39 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -16356,10 +16356,7 @@ static int qemuDomainGetHostnameLease(virDomainObj *vm, char **hostname) { - char macaddr[VIR_MAC_STRING_BUFLEN]; g_autoptr(virConnect) conn = NULL; - virNetworkDHCPLeasePtr *leases = NULL; - int n_leases; size_t i, j; int ret = -1; @@ -16375,6 +16372,9 @@ qemuDomainGetHostnameLease(virDomainObj *vm, for (i = 0; i < vm->def->nnets; i++) { g_autoptr(virNetwork) network = NULL; virDomainNetDef *net = vm->def->nets[i]; + char macaddr[VIR_MAC_STRING_BUFLEN]; + g_autofree virNetworkDHCPLeasePtr *leases = NULL; + int n_leases; if (net->type != VIR_DOMAIN_NET_TYPE_NETWORK) continue; @@ -16390,15 +16390,11 @@ qemuDomainGetHostnameLease(virDomainObj *vm, goto endjob; for (j = 0; j < n_leases; j++) { - virNetworkDHCPLeasePtr lease = leases[j]; + g_autoptr(virNetworkDHCPLease) lease = leases[j]; if (lease->hostname && !*hostname) *hostname = g_strdup(lease->hostname); - - virNetworkDHCPLeaseFree(lease); } - VIR_FREE(leases); - if (*hostname) goto endjob; } -- 2.49.0