Fixes leak 16/26

[modest] / src / modest-formatter.c

diff --git a/src/modest-formatter.c b/src/modest-formatter.c
index f2ecfd9..82418ed 100644 (file)
--- a/src/modest-formatter.c
+++ b/src/modest-formatter.c
@@ -220,6 +220,7 @@ modest_formatter_instance_init (GTypeInstance *instance, gpointer g_class)
        ModestFormatterPrivate *priv = MODEST_FORMATTER_GET_PRIVATE (self);
 
        priv->content_type = NULL;
+       priv->signature = NULL;
 }
 
 static void
@@ -327,7 +328,7 @@ modest_formatter_wrapper_quote (ModestFormatter *self, const gchar *text, TnyHea
        GList *node = NULL;
        gchar *result = NULL;
        gchar *from;
-       
+
        /* First we need a GList of attachments filenames */
        for (node = attachments; node != NULL; node = g_list_next (node)) {
                TnyMimePart *part = (TnyMimePart *) node->data;
@@ -342,12 +343,13 @@ modest_formatter_wrapper_quote (ModestFormatter *self, const gchar *text, TnyHea
                        g_object_unref (header);
                } else {
                        filename = g_strdup (tny_mime_part_get_filename (part));
-                       if ((filename == NULL)||(filename[0] == '\0'))
+                       if ((filename == NULL)||(filename[0] == '\0')) {
+                               g_free (filename);
                                filename = g_strdup ("");
+                       }
                }
-               filenames = g_list_append (filenames, filename);
+               filenames = g_list_prepend (filenames, filename);
        }
-       filenames = g_list_reverse (filenames);
 
        /* TODO: get 80 from the configuration */
        from = tny_header_dup_from (header);