From 18e678c43b5e8bac9c4ca92c09b36dc8f098df48 Mon Sep 17 00:00:00 2001 From: Thomas Vander Stichele Date: Thu, 1 Apr 2004 08:42:30 +0000 Subject: leak plugs and style fixes Original commit message from CVS: leak plugs and style fixes --- ChangeLog | 6 ++++++ gst-libs/gst/gconf/gconf.c | 9 ++++++--- 2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index f6633ad0..0b94ff8b 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2004-04-01 Thomas Vander Stichele + + * gst-libs/gst/gconf/gconf.c: (gst_gconf_get_string), + (gst_gconf_render_bin_from_key): + leak plugging and style fixing + 2004-03-31 David Schleef * gst/audioscale/gstaudioscale.c: (gst_audioscale_expand_value), diff --git a/gst-libs/gst/gconf/gconf.c b/gst-libs/gst/gconf/gconf.c index d68ffe43..00b35a22 100644 --- a/gst-libs/gst/gconf/gconf.c +++ b/gst-libs/gst/gconf/gconf.c @@ -85,7 +85,8 @@ gst_bin_find_unconnected_pad (GstBin * bin, GstPadDirection direction) * * Get GConf key @key's string value. * - * Returns: a #gchar string containing @key's value. + * Returns: a newly allocated #gchar string containing @key's value, + * or NULL in the case of an error.. */ gchar * gst_gconf_get_string (const gchar * key) @@ -99,10 +100,11 @@ gst_gconf_get_string (const gchar * key) g_free (full_key); if (error) { - g_print ("gst_gconf_get_string: error: %s\n", error->message); + g_warning ("gst_gconf_get_string: error: %s\n", error->message); g_error_free (error); + return NULL; } - /* FIXME: decide if we want to strdup this value; if we do, check for NULL */ + return value; } @@ -181,6 +183,7 @@ gst_gconf_render_bin_from_key (const gchar * key) value = gst_gconf_get_string (key); if (value) bin = gst_gconf_render_bin_from_description (value); + g_free (value); return bin; } -- cgit v1.2.1