plug more memory holes

2 files changed, 9 insertions, 3 deletions

diff --git a/connect.c b/connect.c
index 7bc025a..2ea444c 100644
--- a/connect.c
+++ b/connect.c
@@ -58,6 +58,7 @@ static void connect_close(virConnectPtr c, int reason, void *opaque)
 
 static void connect_list(struct vconsole_connect *conn)
 {
+    virDomainPtr d;
     int i, n;
     char **inactive;
     int *active;
@@ -66,7 +67,9 @@ static void connect_list(struct vconsole_connect *conn)
     active = malloc(sizeof(int) * n);
     n = virConnectListDomains(conn->ptr, active, n);
     for (i = 0; i < n; i++) {
-        domain_update(conn, virDomainLookupByID(conn->ptr, active[i]), -1);
+        d = virDomainLookupByID(conn->ptr, active[i]);
+        domain_update(conn, d, -1);
+        virDomainFree(d);
     }
     free(active);
 
@@ -74,7 +77,9 @@ static void connect_list(struct vconsole_connect *conn)
     inactive = malloc(sizeof(char *) * n);
     n = virConnectListDefinedDomains(conn->ptr, inactive, n);
     for (i = 0; i < n; i++) {
-        domain_update(conn, virDomainLookupByName(conn->ptr, inactive[i]), -1);
+        d = virDomainLookupByName(conn->ptr, inactive[i]);
+        domain_update(conn, d, -1);
+        virDomainFree(d);
         free(inactive[i]);
     }
     free(inactive);
diff --git a/vconsole.c b/vconsole.c
index 234fa5f..3660dc7 100644
--- a/vconsole.c
+++ b/vconsole.c
@@ -39,6 +39,7 @@ void config_write(void)
     write(fd, data, len);
     fsync(fd);
     close(fd);
+    g_free(data);
 }
 
 /* ------------------------------------------------------------------ */
@@ -629,7 +630,7 @@ static void vconsole_build_recent(struct vconsole_window *win)
         g_free(h);
         g_free(action);
     }
-    g_free(keys);
+    g_strfreev(keys);
 
     /* finish */
     xml = g_strdup_printf(recent_xml, entries ? entries : "");