notmuch-config: Fix memleaks.
authorHolger Freyther <zecke@selfish.org>
Sat, 21 Nov 2009 21:45:23 +0000 (22:45 +0100)
committerCarl Worth <cworth@cworth.org>
Sat, 21 Nov 2009 23:14:48 +0000 (00:14 +0100)
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>
notmuch-config.c

index 7252a191373c5cc89f3f47df80ebd67766e8946f..aaa0372cd9bd3b4434dfe26e13d29154db260342 100644 (file)
@@ -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;
     }