From: Andrej Kacian <ticho@claws-mail.org>
Date: Fri, 3 Aug 2018 11:22:49 +0000 (+0200)
Subject: Fix memory leaks in toolbar handling of alternate icons (mail/news, ham/spam).
X-Git-Tag: 3.17.0~21
X-Git-Url: http://git.claws-mail.org/?p=claws.git;a=commitdiff_plain;h=e44bccaba2f23878087f5f628a073ce6bf70d17f

Fix memory leaks in toolbar handling of alternate icons (mail/news, ham/spam).
---

diff --git a/src/mainwindow.c b/src/mainwindow.c
index 3762981ed..8e87c709c 100644
--- a/src/mainwindow.c
+++ b/src/mainwindow.c
@@ -3880,8 +3880,7 @@ void main_window_destroy_all(void)
 
 		/* free toolbar stuff */
 		toolbar_clear_list(TOOLBAR_MAIN);
-		TOOLBAR_DESTROY_ACTIONS(mainwin->toolbar->action_list);
-		TOOLBAR_DESTROY_ITEMS(mainwin->toolbar->item_list);
+		toolbar_destroy(mainwin->toolbar);
 
 		summaryview_destroy(mainwin->summaryview);
 		mainwin->folderview->mainwin = NULL;
diff --git a/src/toolbar.c b/src/toolbar.c
index 823a89401..15b05b5f4 100644
--- a/src/toolbar.c
+++ b/src/toolbar.c
@@ -2322,11 +2322,11 @@ Toolbar *toolbar_create(ToolbarType 	 type,
 				_("Compose with selected Account"));
 			toolbar_data->compose_mail_btn = item; 
 			toolbar_data->compose_mail_icon = icon_wid; 
-			g_object_ref(toolbar_data->compose_mail_icon);
+			g_object_ref_sink(toolbar_data->compose_mail_icon);
 
 			icon_news = stock_pixmap_widget(STOCK_PIXMAP_NEWS_COMPOSE);
 			toolbar_data->compose_news_icon = icon_news; 
-			g_object_ref(toolbar_data->compose_news_icon);
+			g_object_ref_sink(toolbar_data->compose_news_icon);
 #else
 			TOOLBAR_ITEM(item,icon_wid,toolbar_item->text,
 				_("Compose Email"));
@@ -2343,11 +2343,11 @@ Toolbar *toolbar_create(ToolbarType 	 type,
 				_("Learn as..."));
 			toolbar_data->learn_spam_btn = item; 
 			toolbar_data->learn_spam_icon = icon_wid; 
-			g_object_ref(toolbar_data->learn_spam_icon);
+			g_object_ref_sink(toolbar_data->learn_spam_icon);
 
 			icon_ham = stock_pixmap_widget(STOCK_PIXMAP_HAM_BTN);
 			toolbar_data->learn_ham_icon = icon_ham; 
-			g_object_ref(toolbar_data->learn_ham_icon);
+			g_object_ref_sink(toolbar_data->learn_ham_icon);
 
 			menu = gtk_menu_new();
 			ADD_MENU_ITEM(_("Learn as _Spam"), toolbar_learn_menu_cb, TRUE);
@@ -2609,13 +2609,22 @@ Toolbar *toolbar_create(ToolbarType 	 type,
 
 /**
  * Free toolbar structures
- */ 
-void toolbar_destroy(Toolbar * toolbar) {
+ */
 
+#define UNREF_ICON(icon) if (toolbar->icon != NULL) \
+			g_object_unref(toolbar->icon)
+
+void toolbar_destroy(Toolbar * toolbar) {
+	UNREF_ICON(compose_mail_icon);
+	UNREF_ICON(compose_news_icon);
+	UNREF_ICON(learn_spam_icon);
+	UNREF_ICON(learn_ham_icon);
 	TOOLBAR_DESTROY_ITEMS(toolbar->item_list);
 	TOOLBAR_DESTROY_ACTIONS(toolbar->action_list);
 }
 
+#undef UNREF_ICON
+
 void toolbar_update(ToolbarType type, gpointer data)
 {
 	Toolbar *toolbar_data;