You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@bugzilla.spamassassin.org on 2007/01/16 12:11:17 UTC

[Bug 5250] 8-bit messages corrupted by rewrite_mail()

http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5250


jm@jmason.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Incorrect behavior with 8-  |8-bit messages corrupted by
                   |bit messages                |rewrite_mail()
   Target Milestone|Undefined                   |3.2.0




------- Additional Comments From jm@jmason.org  2007-01-16 03:11 -------
-1 on that patch...

+		my $msghead = $`;
+		my $msgbody = $';

we cannot use $` and $' in SA -- use of them imposes a speed penalty on the
entire perl interpreter.

however, you're right -- we need to fix a bug here.

The test message contains 8-bit data using Content-Transfer-Encoding: 8bit .
This is a valid MIME message (arbitrary 8-bit data is fine according to rfcs
2045 and 3030).  This is damaged by SpamAssassin due to the s/\r?\n/\r\n/
substitution.

I think your patch is on the right track (replacing \r?\n in the headers only,
rather than in both hdrs+body) -- but it needs to be redone without $` and $'.



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.