[PATCH 2/5] qemuBuildNumaArgStr: Use g_autofree on @nodeBackends

Michal Privoznik mprivozn at redhat.com
Fri May 22 14:14:12 UTC 2020


Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
---
 src/qemu/qemu_command.c | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index 5af22e9359..5a438d07c3 100644
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -7066,13 +7066,13 @@ qemuBuildNumaArgStr(virQEMUDriverConfigPtr cfg,
     size_t i, j;
     virQEMUCapsPtr qemuCaps = priv->qemuCaps;
     g_auto(virBuffer) buf = VIR_BUFFER_INITIALIZER;
-    virBufferPtr nodeBackends = NULL;
+    g_autofree virBufferPtr nodeBackends = NULL;
     bool needBackend = false;
     int ret = -1;
     size_t ncells = virDomainNumaGetNodeCount(def->numa);
 
     if (!virDomainNumatuneNodesetIsAvailable(def->numa, priv->autoNodeset))
-        goto cleanup;
+        return -1;
 
     if (!virQEMUCapsGetMachineNumaMemSupported(qemuCaps,
                                                def->virtType,
@@ -7080,7 +7080,7 @@ qemuBuildNumaArgStr(virQEMUDriverConfigPtr cfg,
         needBackend = true;
 
     if (VIR_ALLOC_N(nodeBackends, ncells) < 0)
-        goto cleanup;
+        return -1;
 
     /* using of -numa memdev= cannot be combined with -numa mem=, thus we
      * need to check which approach to use */
@@ -7159,12 +7159,8 @@ qemuBuildNumaArgStr(virQEMUDriverConfigPtr cfg,
     ret = 0;
 
  cleanup:
-    if (nodeBackends) {
-        for (i = 0; i < ncells; i++)
-            virBufferFreeAndReset(&nodeBackends[i]);
-
-        VIR_FREE(nodeBackends);
-    }
+    for (i = 0; i < ncells; i++)
+        virBufferFreeAndReset(&nodeBackends[i]);
 
     return ret;
 }
-- 
2.26.2




More information about the libvir-list mailing list