From 7323a4e4d86ebad64465ce88a103ee4383af0924 Mon Sep 17 00:00:00 2001 From: kremlin Date: Mon, 21 Jul 2014 19:24:55 -0500 Subject: [PATCH] tighten up memory management in hostnamed.c --- src/interfaces/hostnamed/hostnamed.c | 20 ++++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/src/interfaces/hostnamed/hostnamed.c b/src/interfaces/hostnamed/hostnamed.c index c45905d..1bfcd65 100644 --- a/src/interfaces/hostnamed/hostnamed.c +++ b/src/interfaces/hostnamed/hostnamed.c @@ -24,12 +24,10 @@ #include "hostnamed-gen.h" GPtrArray *hostnamed_freeable; -GDBusNodeInfo *spect_data; Hostname1 *hostnamed_interf; -/* begin method/property/signal code */ +/* --- begin method/property/signal code --- */ -/* TODO make sure these guys only work if called by root */ static gboolean on_handle_set_hostname(Hostname1 *hn1_passed_interf, GDBusMethodInvocation *invoc, @@ -79,11 +77,13 @@ on_handle_set_icon_name(Hostname1 *hn1_passed_interf, const gchar * our_get_hostname() { - int hostname_try; gchar *hostname_buf; hostname_buf = (gchar*) g_malloc0(MAXHOSTNAMELEN); - hostname_try = gethostname(hostname_buf, MAXHOSTNAMELEN); + g_ptr_array_add(hostnamed_freeable, hostname_buf); + + if(gethostname(hostname_buf, MAXHOSTNAMELEN)) + return ""; return hostname_buf; } @@ -142,7 +142,7 @@ our_get_os_pretty_name() { return "TODO"; } -/* end method/property/signal code, begin bus/name handlers */ +/* --- end method/property/signal code, begin bus/name handlers --- */ static void hostnamed_on_bus_acquired(GDBusConnection *conn, const gchar *name, @@ -189,10 +189,13 @@ static void hostnamed_on_name_acquired(GDBusConnection *conn, } +/* --- end bus/name handlers, begin misc functions --- */ + /* free()'s */ void hostnamed_mem_clean() { - g_ptr_array_foreach(hostnamed_freeable, (GFunc) g_free, NULL); + ddg_ptr_array_foreach(hostnamed_freeable, (GFunc) g_free, NULL); + g_ptr_array_free(hostnamed_freeable); } static void hostnamed_on_name_lost(GDBusConnection *conn, @@ -229,11 +232,12 @@ int main() { /* config stuff here */ - hostnamed_init(); g_main_loop_run(hostnamed_loop); g_main_loop_unref(hostnamed_loop); + hostnamed_mem_clean(); + return 0; } -- 2.41.0