]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
virCapabilitiesHostNUMAInitReal: Bring variables into loop
authorMichal Privoznik <mprivozn@redhat.com>
Tue, 4 May 2021 07:47:39 +0000 (09:47 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Mon, 10 May 2021 13:16:35 +0000 (15:16 +0200)
Some variables are needed only inside for() loop. They were
declared at the beginning of the function because of VIR_FREE()
calls, but since they are auto-freed they can be declared inside
the loop.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
src/conf/capabilities.c

index c487229ae892d1e023c8a0099081f976fb0bec35..1dae6d38cc0309f566e0bbe6ac29a0c0d4cb107d 100644 (file)
@@ -1596,22 +1596,22 @@ static int
 virCapabilitiesHostNUMAInitReal(virCapsHostNUMA *caps)
 {
     int n;
-    unsigned long long memory;
     virCapsHostNUMACellCPU *cpus = NULL;
-    g_autoptr(virBitmap) cpumap = NULL;
-    g_autofree virCapsHostNUMACellSiblingInfo *siblings = NULL;
-    int nsiblings = 0;
-    g_autofree virCapsHostNUMACellPageInfo *pageinfo = NULL;
-    int npageinfo;
     int ret = -1;
     int ncpus = 0;
-    int cpu;
     int max_node;
 
     if ((max_node = virNumaGetMaxNode()) < 0)
         goto cleanup;
 
     for (n = 0; n <= max_node; n++) {
+        g_autoptr(virBitmap) cpumap = NULL;
+        g_autofree virCapsHostNUMACellSiblingInfo *siblings = NULL;
+        int nsiblings = 0;
+        g_autofree virCapsHostNUMACellPageInfo *pageinfo = NULL;
+        int npageinfo;
+        unsigned long long memory;
+        int cpu;
         size_t i;
 
         if ((ncpus = virNumaGetNodeCPUs(n, &cpumap)) < 0) {
@@ -1645,8 +1645,6 @@ virCapabilitiesHostNUMAInitReal(virCapsHostNUMA *caps)
                                        ncpus, &cpus,
                                        nsiblings, &siblings,
                                        npageinfo, &pageinfo);
-        virBitmapFree(cpumap);
-        cpumap = NULL;
     }
 
     ret = 0;