[PATCH 02/36] conf: capabilities: Clean up freeing of virBitmap

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Use automatic freeing where possible and use g_clear_pointer instead of
manual NULL-ing.

Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
---
 src/conf/capabilities.c | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/src/conf/capabilities.c b/src/conf/capabilities.c
index 34295ec4d7..ef594d7241 100644
--- a/src/conf/capabilities.c
+++ b/src/conf/capabilities.c
@@ -104,10 +104,8 @@ virCapabilitiesClearHostNUMACellCPUTopology(virCapsHostNUMACellCPU *cpus,
     if (!cpus)
         return;

-    for (i = 0; i < ncpus; i++) {
-        virBitmapFree(cpus[i].siblings);
-        cpus[i].siblings = NULL;
-    }
+    for (i = 0; i < ncpus; i++)
+        g_clear_pointer(&cpus[i].siblings, virBitmapFree);
 }

 static void
@@ -1416,20 +1414,18 @@ virBitmap *
 virCapabilitiesHostNUMAGetCpus(virCapsHostNUMA *caps,
                                virBitmap *nodemask)
 {
-    virBitmap *ret = NULL;
+    g_autoptr(virBitmap) ret = NULL;
     unsigned int maxcpu = virCapabilitiesHostNUMAGetMaxcpu(caps);
     ssize_t node = -1;

     ret = virBitmapNew(maxcpu + 1);

     while ((node = virBitmapNextSetBit(nodemask, node)) >= 0) {
-        if (virCapabilitiesHostNUMAGetCellCpus(caps, node, ret) < 0) {
-            virBitmapFree(ret);
+        if (virCapabilitiesHostNUMAGetCellCpus(caps, node, ret) < 0)
             return NULL;
-        }
     }

-    return ret;
+    return g_steal_pointer(&ret);
 }


-- 
2.31.1




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux