commit d4c8a5d49e514bfeac2040ee5371b4e6a7d8d561
author Marc-André Lureau <marcandre.lureau@redhat.com> Thu Aug 27 01:34:56 2015 +0200
committer Michael Roth <mdroth@linux.vnet.ibm.com> Tue Sep 01 13:16:22 2015 -0500
tree 625c00948a8ba4b4323c603b20f9cb074c84c5fd
parent e3d3103975112a1ab8a0129a4be1cfe3314bce8b

qga: free a bit more

Now that main() has a single exit point, we can free a few
more allocations.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Denis V. Lunev <den@openvz.org>
Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>

qga/main.c

diff --git a/qga/main.c b/qga/main.c
index b5859bb..fc07696 100644
--- a/qga/main.c
+++ b/qga/main.c
@@ -82,7 +82,7 @@
     bool delimit_response;
     bool frozen;
     GList *blacklist;
-    const char *state_filepath_isfrozen;
+    char *state_filepath_isfrozen;
     struct {
         const char *log_filepath;
         const char *pid_filepath;
@@ -90,7 +90,7 @@
 #ifdef CONFIG_FSFREEZE
     const char *fsfreeze_hook;
 #endif
-    const gchar *pstate_filepath;
+    gchar *pstate_filepath;
     GAPersistentState pstate;
 };
 
@@ -1254,6 +1254,8 @@
         ga_channel_free(s->channel);
     }
     g_list_foreach(config->blacklist, free_blacklist_entry, NULL);
+    g_free(s->pstate_filepath);
+    g_free(s->state_filepath_isfrozen);
 
     if (config->daemonize) {
         unlink(config->pid_filepath);