You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by fe...@apache.org on 2004/09/05 19:40:26 UTC
svn commit: rev 43393 - spamassassin/trunk/lib/Mail/SpamAssassin
Author: felicity
Date: Sun Sep 5 10:40:25 2004
New Revision: 43393
Modified:
spamassassin/trunk/lib/Mail/SpamAssassin/Message.pm
Log:
revert of the bug 3749 patch. fixes one issue, made a new set. :(
Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Message.pm
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Message.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Message.pm Sun Sep 5 10:40:25 2004
@@ -469,18 +469,13 @@
if ($part_array) {
chomp( $part_array->[-1] ); # trim the CRLF that's part of the boundary
splice @{$part_array}, -1 if ( $part_array->[-1] eq '' ); # blank line for the boundary only ...
- }
- else {
- # Invalid parts can have no body, so fake in a blank body
- # in that case.
- $part_array = [];
- }
- my($p_boundary);
- ($part_msg->{'type'}, $p_boundary) = Mail::SpamAssassin::Util::parse_content_type($part_msg->header('content-type'));
- $p_boundary ||= $boundary;
- dbg("found part of type ".$part_msg->{'type'}.", boundary: ".(defined $p_boundary ? $p_boundary : ''));
- $self->parse_body( $msg, $part_msg, $p_boundary, $part_array, 0 );
+ my($p_boundary);
+ ($part_msg->{'type'}, $p_boundary) = Mail::SpamAssassin::Util::parse_content_type($part_msg->header('content-type'));
+ $p_boundary ||= $boundary;
+ dbg("found part of type ".$part_msg->{'type'}.", boundary: ".(defined $p_boundary ? $p_boundary : ''));
+ $self->parse_body( $msg, $part_msg, $p_boundary, $part_array, 0 );
+ }
if (defined $boundary && $line =~ /^\-\-\Q${boundary}\E\-\-$/) {
# Make a note that we've seen the end boundary