I am not 100% sure on this one, but abrt reported a crash in line 203. The value of mi was optimized out and there was no debugging info from the causing function (in general abrt seems to be far away from really usefull), but since mi _can_ be NULL here and the get_flags operation derefernces this pointer without further checking, I guess this is the correct solution.
Signed-off-by: Christoph Höger <[email protected]> --- camel/providers/local/camel-maildir-folder.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/camel/providers/local/camel-maildir-folder.c b/camel/providers/local/camel-maildir-folder.c index d4c9912..c13207d 100644 --- a/camel/providers/local/camel-maildir-folder.c +++ b/camel/providers/local/camel-maildir-folder.c @@ -197,7 +197,7 @@ maildir_append_message (CamelFolder *folder, CamelMimeMessage *message, const Ca /* add it to the summary/assign the uid, etc */ mi = camel_local_summary_add((CamelLocalSummary *)folder->summary, message, info, lf->changes, ex); - if (camel_exception_is_set (ex)) + if (!mi || camel_exception_is_set (ex)) goto check_changed; if ((camel_message_info_flags (mi) & CAMEL_MESSAGE_ATTACHMENTS) && !camel_mime_message_has_attachment (message)) -- 1.6.5.2 _______________________________________________ Evolution-hackers mailing list [email protected] http://mail.gnome.org/mailman/listinfo/evolution-hackers
