Closed Bug 286447 Opened 20 years ago Closed 19 years ago

Strip attachment (detach/delete) from message with odd quoting breaks MIME structure on rewrite

Categories

(MailNews Core :: Attachments, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mcow, Assigned: sspitzer)

Details

Attachments

(1 file)

Error with bug 2920's patch.  TB 1.0+0316, Windows 2000.

Given a plain message body which (incorrectly?) has an empty but quote-prefixed 
line immediately before the attachment's MIME boundary, if the attachment is 
stripped, the rewritten message is incorrect.  Specifically in this case, the 
boundary signifying the end of the message body is written onto the same line as 
the final (empty) quotemarks.  This causes the 'attachment/detachment' part to 
be treated as part of the message body.

  multipart/related; boundary=XXX
    --XXX
    text/plain
    blah
    >>quote
    >>
    --XXX
    application/x-whatever
    ....
    --XXX--

becomes
  multipart/related; boundary=XXX
    --XXX
    text/plain
    blah
    >>quote
    >>--XXX
    text/x-moz-deleted ...
    --XXX--

For a detachment, this has the effect of losing the capability to load the 
document.
Attached file test case
Here's a sample to test with.
Steps to reproduce:
1) integrate this message into your mail system
2) View the message; delete and/or detach the attachment
3 [details] [diff] [review]) View the resulting message

Actual results:
3) Message body displayed includes the MIME boundary as the last line of the
quoted text, and then shows the moz-text-deleted headers and text inline; empty
attachment panel.

Expected results:
3) Message body unaltered; moz-text-deleted shown, at most, as inline
attachment data; item shown in panel.
This bug was fixed by the patch in attachment 179215 [details] [diff] [review] -- which was posted at the 
wrong bug, bug 286446.  Thanks, David!
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: