summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEduardo Chappa <chappa@washington.edu>2021-02-25 21:46:18 -0700
committerEduardo Chappa <chappa@washington.edu>2021-02-25 21:46:18 -0700
commit872e50a5db8c27b1cf24b1bfadde8cfc89ade905 (patch)
treebe309a39837351ccb2277b4b0294c39059ca0fb8
parentaa6f4ec5bfdd58483b92941a8494b8edc6aa0320 (diff)
downloadalpine-872e50a5db8c27b1cf24b1bfadde8cfc89ade905.tar.xz
* Fix: When deleting an attachment, saving the message with the deleted
attachment to the mailbox, will add extra null characters.
-rw-r--r--pith/pine.hlp4
-rw-r--r--pith/save.c1
2 files changed, 5 insertions, 0 deletions
diff --git a/pith/pine.hlp b/pith/pine.hlp
index 26e5e26..6ab8a33 100644
--- a/pith/pine.hlp
+++ b/pith/pine.hlp
@@ -191,6 +191,10 @@ Bugs addressed:
<UL>
<LI> When forwarding a message, replacing an attachment might make Alpine
re-attach the original attachment. Reported by Michael Traxler.
+
+<LI> When an attachment is deleted, the saved message with the deleted
+attachment contains extra null characters after the end of the
+attachment boundary.
</UL>
<P> Version 2.24 adds new features and addresses bugs found in previous
diff --git a/pith/save.c b/pith/save.c
index 2d74d54..fe7f974 100644
--- a/pith/save.c
+++ b/pith/save.c
@@ -1602,6 +1602,7 @@ save_ex_explain_body(struct mail_bodystruct *body, long unsigned int *len, gf_io
else
return(0);
+ ilen = 0; /* maybe save_ex_explain_parts should set this to zero instead */
if(!save_ex_explain_parts(body, 0, &ilen, pc))
return(0);