[libvirt PATCH 2/5] vsh: cmdEcho: use separate variable for argument

Ján Tomko jtomko at redhat.com
Tue Aug 10 17:14:40 UTC 2021


Do not use 'arg' which is later used for an allocated string.

Signed-off-by: Ján Tomko <jtomko at redhat.com>
---
 tools/vsh.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/tools/vsh.c b/tools/vsh.c
index 3bbaecd2ea..bf32a8dc22 100644
--- a/tools/vsh.c
+++ b/tools/vsh.c
@@ -3199,17 +3199,16 @@ cmdEcho(vshControl *ctl, const vshCmd *cmd)
     while ((opt = vshCommandOptArgv(ctl, cmd, opt))) {
         char *str;
         g_auto(virBuffer) xmlbuf = VIR_BUFFER_INITIALIZER;
-
-        arg = opt->data;
+        const char *curr = opt->data;
 
         if (count)
             virBufferAddChar(&buf, ' ');
 
         if (xml) {
-            virBufferEscapeString(&xmlbuf, "%s", arg);
+            virBufferEscapeString(&xmlbuf, "%s", curr);
             str = virBufferContentAndReset(&xmlbuf);
         } else {
-            str = g_strdup(arg);
+            str = g_strdup(curr);
         }
 
         if (shell)
-- 
2.31.1




More information about the libvir-list mailing list