Bug 516 - Carriage return problems
Summary: Carriage return problems
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: Libraries (show other bugs)
Version: 2.31
Hardware: Other other
: P2 critical
Target Milestone: ---
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-07-04 11:06 UTC by Daniel Rogers
Modified: 2002-08-07 21:33 UTC (History)
0 users


Attachment Type Modified Status Actions Submitter/CLA Status
Patch to remove newlines patch None Daniel Rogers [NoCLA]
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Rogers 2002-07-04 11:06:26 UTC 
In PerMsgStatus.pm, mime parts aren't recognized properly if lines end with \r\n
(as with MimeDefang messages).  This patch strips the CRs for processing.

Dan.
Comment 1 Daniel Rogers 2002-07-04 11:06:49 UTC 
Created attachment 209 [details]
Patch to remove newlines
Comment 2 Daniel Rogers 2002-07-04 11:07:19 UTC 
Oh, without this patch, PORN_10,UPPERCASE_25_50,SUPERLONG_LINE can all easily be
tripped by attachments.

Dan.
Comment 3 Daniel Rogers 2002-08-07 09:09:32 UTC 
Increasing severity and reassigning to the list.

This should make it in to 2.40, please.

(See bug 398 as well.  Should we be stripping \r's while reading the message in?)

Dan.
Comment 4 Justin Mason 2002-08-08 05:33:43 UTC 
ok, checked in