[Commits] [SCM] claws branch, master, updated. 3.15.0-68-gd0cd541

[mones at claws-mail.org]

The branch, master has been updated
       via  d0cd54152fa784f31fa3d01e80cc4f20eb2d2d6f (commit)
      from  c8e4c10674d1abee12764bf67fc0bda2e4a4fea9 (commit)

Summary of changes:
 src/plugins/att_remover/att_remover.c |    9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)


- Log -----------------------------------------------------------------
commit d0cd54152fa784f31fa3d01e80cc4f20eb2d2d6f
Author: Ricardo Mones <ricardo at mones.org>
Date:   Sat Jun 17 17:05:55 2017 +0200

    Fix bug 3822 ‘AttRemover deletes message and fails to create new one when disk is full’

diff --git a/src/plugins/att_remover/att_remover.c b/src/plugins/att_remover/att_remover.c
index 1092c15..0946bb1 100644
--- a/src/plugins/att_remover/att_remover.c
+++ b/src/plugins/att_remover/att_remover.c
@@ -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);

-----------------------------------------------------------------------


hooks/post-receive
-- 
Claws Mail