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