]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
qemuBuildNumaCommandLine: Move vars into loops
authorMichal Privoznik <mprivozn@redhat.com>
Wed, 20 May 2020 09:32:14 +0000 (11:32 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Fri, 1 Oct 2021 08:52:35 +0000 (10:52 +0200)
There are two variables that are used only in a single
loop. Move their definitions into their respective blocks.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Peter Krempa <pkrempa@redhat.com>
src/qemu/qemu_command.c

index 48df8818a6bf515c314795b209420d33674f3e63..247ac86e33f56d7509b8f5d7c646a56772c5fb22 100644 (file)
@@ -7290,11 +7290,9 @@ qemuBuildNumaCommandLine(virQEMUDriverConfig *cfg,
     size_t i, j;
     virQEMUCaps *qemuCaps = priv->qemuCaps;
     g_auto(virBuffer) buf = VIR_BUFFER_INITIALIZER;
-    char *next = NULL;
     virBuffer *nodeBackends = NULL;
     bool needBackend = false;
     bool hmat = false;
-    int rc;
     int ret = -1;
     size_t ncells = virDomainNumaGetNodeCount(def->numa);
     ssize_t masterInitiator = -1;
@@ -7319,6 +7317,7 @@ qemuBuildNumaCommandLine(virQEMUDriverConfig *cfg,
     if (virQEMUCapsGet(qemuCaps, QEMU_CAPS_OBJECT_MEMORY_RAM) ||
         virQEMUCapsGet(qemuCaps, QEMU_CAPS_OBJECT_MEMORY_FILE) ||
         virQEMUCapsGet(qemuCaps, QEMU_CAPS_OBJECT_MEMORY_MEMFD)) {
+        int rc;
 
         for (i = 0; i < ncells; i++) {
             if ((rc = qemuBuildMemoryCellBackendStr(def, cfg, i, priv,
@@ -7361,6 +7360,7 @@ qemuBuildNumaCommandLine(virQEMUDriverConfig *cfg,
 
         if (cpumask) {
             g_autofree char *cpumaskStr = NULL;
+            char *next = NULL;
             char *tmpmask;
 
             if (!(cpumaskStr = virBitmapFormat(cpumask)))