From: Dirk-Jan C. Binnema <dirk-jan.binnema@nokia.com>
Date: Mon, 5 Nov 2007 14:53:12 +0000 (+0000)
Subject: * some extra error handling to prevent NULL dereferences
X-Git-Tag: git_migration_finished~2163
X-Git-Url: http://git.maemo.org/git/?p=modest;a=commitdiff_plain;h=2f0432fdff3a5230e1779dd11518a65f1544bb04;hp=a121dd4c6adfb3a13a3cf03260a49207dfd0fee7;ds=sidebyside

* some extra error handling to prevent NULL dereferences

pmo-trunk-r3648
---

diff --git a/src/modest-ui-actions.c b/src/modest-ui-actions.c
index 1943ffa..9cffa20 100644
--- a/src/modest-ui-actions.c
+++ b/src/modest-ui-actions.c
@@ -4196,10 +4196,14 @@ modest_ui_actions_on_main_window_remove_attachments (GtkAction *action,
 		iter = tny_list_create_iterator (header_list);
 		header = TNY_HEADER (tny_iterator_get_current (iter));
 		g_object_unref (iter);
-	} else {
+	} else
+		return;
+	
+	if (!header || !TNY_IS_HEADER(header)) {
+		g_warning ("%s: header is not valid", __FUNCTION__);
 		return;
 	}
-
+	
 	found = modest_window_mgr_find_registered_header (modest_runtime_get_window_mgr (),
 							  header, &msg_view_window);
 	flags = tny_header_get_flags (header);