]> xenbits.xensource.com Git - libvirt.git/commitdiff
virLockDaemonPostExecRestart: Automatically free temporary variables
authorPeter Krempa <pkrempa@redhat.com>
Wed, 1 Dec 2021 08:23:19 +0000 (09:23 +0100)
committerPeter Krempa <pkrempa@redhat.com>
Tue, 7 Dec 2021 08:00:39 +0000 (09:00 +0100)
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>
src/locking/lock_daemon.c

index fc714052be812153337c83fc4caf5d2b8792664b..1edd912c3e8bf3940b7edf5d26275fa1724a22da 100644 (file)
@@ -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;
 }