[notmuch] [PATCH] notmuch-config: Fix memleaks.
Stefan Schmidt
stefan at datenfreihafen.org
Sat Nov 21 14:17:02 PST 2009
>From 4bb598e6e0fa32d5184be54cb5c66fe364eba374 Mon Sep 17 00:00:00 2001
From: Holger Freyther <zecke at selfish.org>
Date: Sat, 21 Nov 2009 22:45:23 +0100
Subject: [PATCH] notmuch-config: Fix memleaks.
While talloc is great we need to free the g_error by hand.
Tested-by: Stefan Schmidt <stefan at datenfreihafen.org>
Signed-off-by: Holger Freyther <zecke at selfish.org>
---
notmuch-config.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/notmuch-config.c b/notmuch-config.c
index 7252a19..aaa0372 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;
}
--
1.6.5.3
More information about the notmuch
mailing list