From: Ján Tomko <jtomko@redhat.com>
Date: Mon, 18 May 2020 13:23:09 +0000 (+0200)
Subject: qemu: do not leak the directory in virQEMUDriverConfigLoadMemoryEntry
X-Git-Tag: v6.4.0-rc1~30
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=8da8cafdcb8457a5c3869d347fffb1f69c2d7a04;p=thirdparty%2Flibvirt.git

qemu: do not leak the directory in virQEMUDriverConfigLoadMemoryEntry

virConfGetValueString returns an allocated string that needs to be
freed.

Fixes: 34a59fb570dcc1b6b4b4cef1f0eb43fc6f688120
Signed-off-by: Ján Tomko <jtomko@redhat.com>
Reviewed-by: Daniel Henrique Barboza <danielhb413@gmail.com>
---

diff --git a/src/qemu/qemu_conf.c b/src/qemu/qemu_conf.c
index 2d5e527b47..2d4e2a46f3 100644
--- a/src/qemu/qemu_conf.c
+++ b/src/qemu/qemu_conf.c
@@ -969,7 +969,7 @@ static int
 virQEMUDriverConfigLoadMemoryEntry(virQEMUDriverConfigPtr cfg,
                                    virConfPtr conf)
 {
-    char *dir = NULL;
+    g_autofree char *dir = NULL;
     int rc;
 
     if ((rc = virConfGetValueString(conf, "memory_backing_dir", &dir)) < 0) {