From: Peter Krempa <pkrempa@redhat.com>
Date: Wed, 1 Dec 2021 08:23:19 +0000 (+0100)
Subject: virLockDaemonPostExecRestart: Automatically free temporary variables
X-Git-Tag: v8.0.0-rc1~367
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=bec4f09fff3620be431fd503c9128a1c94a538f4;p=thirdparty%2Flibvirt.git

virLockDaemonPostExecRestart: Automatically free temporary variables

Convert two temp strings and one virJSONValue to g_auto(free|ptr).

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
---

diff --git a/src/locking/lock_daemon.c b/src/locking/lock_daemon.c
index fc714052be..1edd912c3e 100644
--- a/src/locking/lock_daemon.c
+++ b/src/locking/lock_daemon.c
@@ -607,10 +607,10 @@ virLockDaemonPostExecRestart(const char *state_file,
                              bool privileged)
 {
     const char *gotmagic;
-    char *wantmagic = NULL;
+    g_autofree char *wantmagic = NULL;
     int ret = -1;
-    char *state = NULL;
-    virJSONValue *object = NULL;
+    g_autofree char *state = NULL;
+    g_autoptr(virJSONValue) object = NULL;
 
     VIR_DEBUG("Running post-restart exec");
 
@@ -660,9 +660,6 @@ virLockDaemonPostExecRestart(const char *state_file,
 
  cleanup:
     unlink(state_file);
-    VIR_FREE(wantmagic);
-    VIR_FREE(state);
-    virJSONValueFree(object);
     return ret;
 }