Return-Path: Delivered-To: apmail-spamassassin-dev-archive@www.apache.org Received: (qmail 38605 invoked from network); 4 Sep 2004 21:30:42 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur-2.apache.org with SMTP; 4 Sep 2004 21:30:42 -0000 Received: (qmail 42177 invoked by uid 500); 4 Sep 2004 21:30:41 -0000 Delivered-To: apmail-spamassassin-dev-archive@spamassassin.apache.org Received: (qmail 42155 invoked by uid 500); 4 Sep 2004 21:30:41 -0000 Mailing-List: contact dev-help@spamassassin.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: list-post: List-Id: "SpamAssassin Development" dev@spamassassin.apache.org Delivered-To: mailing list dev@spamassassin.apache.org Received: (qmail 42142 invoked by uid 99); 4 Sep 2004 21:30:41 -0000 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests=NO_REAL_NAME,SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (hermes.apache.org: domain of apache@bugzilla.spamassassin.org designates 64.142.3.173 as permitted sender) Received: from [64.142.3.173] (HELO bugzilla.spamassassin.org) (64.142.3.173) by apache.org (qpsmtpd/0.28) with ESMTP; Sat, 04 Sep 2004 14:30:41 -0700 Received: by bugzilla.spamassassin.org (Postfix, from userid 48) id CA361838A9; Sat, 4 Sep 2004 14:30:39 -0700 (PDT) From: bugzilla-daemon@bugzilla.spamassassin.org To: dev@spamassassin.apache.org Subject: [Bug 3749] message parser skips blank/invalid(?) parts X-Bugzilla-Reason: CC Message-Id: <20040904213039.CA361838A9@bugzilla.spamassassin.org> Date: Sat, 4 Sep 2004 14:30:39 -0700 (PDT) X-Virus-Checked: Checked X-Spam-Rating: minotaur-2.apache.org 1.6.2 0/1000/N http://bugzilla.spamassassin.org/show_bug.cgi?id=3749 ------- Additional Comments From felicity@kluge.net 2004-09-04 14:30 ------- Created an attachment (id=2315) --> (http://bugzilla.spamassassin.org/attachment.cgi?id=2315&action=view) suggested patch mostly does the same thing as before. the original code specifically disregarded parts in a multipart that had no body. the new version makes the assumption that if there is no body, a blank one should be assumed and the part added. the second section (p_boundary ... parse_body) is the same as it was except for whitespace changing. that block adds the part appropriately into the tree. the meat is in the first section where the main change is that if $part_array isn't true (which only happens when it's not defined since it's an array reference), we should generate a reference to an empty array as the part body. passes "make test" for me ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.