From 16c1a13ce46a9befd8fbe3d0fd63ce80c1a5cd97 Mon Sep 17 00:00:00 2001 From: Jose Dapena Paz Date: Sat, 5 Sep 2009 12:52:46 +0200 Subject: [PATCH] Code review fix in ModestAttachmentsView. * src/widgets/modest-attachments-view.c (modest_attachments_view_set_message): don't assume msg_content_type != NULL doing an strcasecmp call (fix 1/2 NB#136381). --- src/widgets/modest-attachments-view.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/widgets/modest-attachments-view.c b/src/widgets/modest-attachments-view.c index 8d498f3..e7d5a5c 100644 --- a/src/widgets/modest-attachments-view.c +++ b/src/widgets/modest-attachments-view.c @@ -159,7 +159,7 @@ modest_attachments_view_set_message (ModestAttachmentsView *attachments_view, Tn part_to_check = modest_tny_msg_get_attachments_parent (TNY_MSG (msg)); msg_content_type = modest_tny_mime_part_get_content_type (TNY_MIME_PART (part_to_check)); - is_alternate = !strcasecmp (msg_content_type, "multipart/alternative"); + is_alternate = (msg_content_type != NULL) && !strcasecmp (msg_content_type, "multipart/alternative"); /* If the top mime part is a multipart/related, we don't show the attachments, as they're * embedded images in body */ -- 1.7.9.5