From: Holger Freyther <zecke@selfish.org>
Date: Sat, 21 Nov 2009 21:45:23 +0000 (+0100)
Subject: notmuch-config: Fix memleaks.
X-Git-Tag: 0.1~378
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=6c7ec294bbe5ae4c3e1498b5061103f08076e016;p=notmuch.git

notmuch-config: Fix memleaks.

While talloc is great we need to free the g_error by hand.

Tested-by: Stefan Schmidt <stefan@datenfreihafen.org>
Signed-off-by: Holger Freyther <zecke@selfish.org>
---

diff --git a/notmuch-config.c b/notmuch-config.c
index 7252a191..aaa0372c 100644
--- a/notmuch-config.c
+++ b/notmuch-config.c
@@ -207,9 +207,11 @@ notmuch_config_open (void *ctx,
 	    fprintf (stderr, "Error reading configuration file %s: %s\n",
 		     config->filename, error->message);
 	    talloc_free (config);
+	    g_error_free (error);
 	    return NULL;
 	}
 
+	g_error_free (error);
 	is_new = 1;
     }
 
@@ -308,6 +310,7 @@ notmuch_config_save (notmuch_config_t *config)
     if (! g_file_set_contents (config->filename, data, length, &error)) {
 	fprintf (stderr, "Error saving configuration to %s: %s\n",
 		 config->filename, error->message);
+	g_error_free (error);
 	return 1;
     }