Fix bug 3822 ‘AttRemover deletes message and fails to create new one when disk is...
authorRicardo Mones <ricardo@mones.org>
Sat, 17 Jun 2017 15:05:55 +0000 (17:05 +0200)
committerRicardo Mones <ricardo@mones.org>
Sat, 17 Jun 2017 15:05:55 +0000 (17:05 +0200)
src/plugins/att_remover/att_remover.c

index 1092c15..0946bb1 100644 (file)
@@ -129,9 +129,14 @@ static gint save_new_message(MsgInfo *oldmsg, MsgInfo *newmsg, MimeInfo *info,
                flags.tmp_flags &= ~MSG_HAS_ATTACHMENT;
 
        oldmsg->flags.perm_flags &= ~MSG_LOCKED;
+       msgnum = folder_item_add_msg(item, finalmsg->plaintext_file, &flags, TRUE);
+       if (msgnum < 0) {
+               g_warning("could not add message without attachments");
+               procmsg_msginfo_free(&newmsg);
+               procmsg_msginfo_free(&finalmsg);
+               return msgnum;
+       }
        folder_item_remove_msg(item, oldmsg->msgnum);
-       msgnum = folder_item_add_msg(item, finalmsg->plaintext_file, 
-                       &flags, TRUE);
        finalmsg->msgnum = msgnum;
        procmsg_msginfo_free(&newmsg);
        procmsg_msginfo_free(&finalmsg);