fix one of the earlier memleaks reported by Martin Kluge
authorAlfons Hoogervorst <alfons@proteus.demon.nl>
Mon, 19 Aug 2002 20:56:50 +0000 (20:56 +0000)
committerAlfons Hoogervorst <alfons@proteus.demon.nl>
Mon, 19 Aug 2002 20:56:50 +0000 (20:56 +0000)
ChangeLog.claws
configure.in
src/summaryview.c

index 235fc50..a3fe154 100644 (file)
@@ -1,3 +1,8 @@
+2002-08-19 [alfons]    0.8.1claws80
+
+       * src/summaryview.c
+               fix one of the earlier memleaks reported by Martin Kluge
+
 2002-08-19 [alfons]    0.8.1claws79
 
        * src/mainwindow.c
index 8b1b278..91f3881 100644 (file)
@@ -8,7 +8,7 @@ MINOR_VERSION=8
 MICRO_VERSION=1
 INTERFACE_AGE=0
 BINARY_AGE=0
-EXTRA_VERSION=claws79
+EXTRA_VERSION=claws80
 VERSION=$MAJOR_VERSION.$MINOR_VERSION.$MICRO_VERSION$EXTRA_VERSION
 
 dnl set $target
index 27aee33..ad415ff 100644 (file)
@@ -2074,6 +2074,7 @@ static void summary_set_header(SummaryView *summaryview, gchar *text[],
 {
        static gchar date_modified[80];
        static gchar *to = NULL;
+       static gchar *from = NULL;
        static gchar col_score[11];
        static gchar buf[BUFFSIZE];
        gint *col_pos = summaryview->col_pos;
@@ -2127,16 +2128,10 @@ static void summary_set_header(SummaryView *summaryview, gchar *text[],
         * the --> in sent boxes) was executed.
         */
        if (text[col_pos[S_COL_FROM]] != to && prefs_common.use_addr_book && msginfo->from) {
-               gchar *from;
+               g_free(from);
                from = summary_complete_address(msginfo->from);
-               if (from) {
-                       /*
-                        * FIXME: this text[col_pos[S_COL_FROM]] should be freed
-                        * but may have been assigned _("No From"). Should be
-                        * freed??? 
-                        */
+               if (from)
                        text[col_pos[S_COL_FROM]] = from;
-               }       
        }
 
        if (summaryview->simplify_subject_preg != NULL)