Return-Path: Delivered-To: apmail-geronimo-scm-archive@www.apache.org Received: (qmail 69907 invoked from network); 25 Aug 2008 12:54:06 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 25 Aug 2008 12:54:06 -0000 Received: (qmail 53840 invoked by uid 500); 25 Aug 2008 12:54:04 -0000 Delivered-To: apmail-geronimo-scm-archive@geronimo.apache.org Received: (qmail 53779 invoked by uid 500); 25 Aug 2008 12:54:04 -0000 Mailing-List: contact scm-help@geronimo.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: List-Post: Reply-To: dev@geronimo.apache.org List-Id: Delivered-To: mailing list scm@geronimo.apache.org Received: (qmail 53769 invoked by uid 99); 25 Aug 2008 12:54:04 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 25 Aug 2008 05:54:04 -0700 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.4] (HELO eris.apache.org) (140.211.11.4) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 25 Aug 2008 12:53:15 +0000 Received: by eris.apache.org (Postfix, from userid 65534) id 4701A2388970; Mon, 25 Aug 2008 05:53:15 -0700 (PDT) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r688710 - in /geronimo/specs/trunk/geronimo-javamail_1.4_spec: pom.xml src/main/java/javax/mail/internet/InternetHeaders.java Date: Mon, 25 Aug 2008 12:53:14 -0000 To: scm@geronimo.apache.org From: rickmcguire@apache.org X-Mailer: svnmailer-1.0.8 Message-Id: <20080825125315.4701A2388970@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org Author: rickmcguire Date: Mon Aug 25 05:53:14 2008 New Revision: 688710 URL: http://svn.apache.org/viewvc?rev=688710&view=rev Log: GERONIMO-4261 MimeMessage throws IndexOutOfBoundsException on malformed data Modified: geronimo/specs/trunk/geronimo-javamail_1.4_spec/pom.xml geronimo/specs/trunk/geronimo-javamail_1.4_spec/src/main/java/javax/mail/internet/InternetHeaders.java Modified: geronimo/specs/trunk/geronimo-javamail_1.4_spec/pom.xml URL: http://svn.apache.org/viewvc/geronimo/specs/trunk/geronimo-javamail_1.4_spec/pom.xml?rev=688710&r1=688709&r2=688710&view=diff ============================================================================== --- geronimo/specs/trunk/geronimo-javamail_1.4_spec/pom.xml (original) +++ geronimo/specs/trunk/geronimo-javamail_1.4_spec/pom.xml Mon Aug 25 05:53:14 2008 @@ -47,9 +47,8 @@ specs/${artifactId} - javax.mail* - org.apache.geronimo.mail* - 1.4 + javax.mail*,org.apache.geronimo.mail* + 1.5-SNAPSHOT src/main/resources/,META-INF/LICENSE.txt=LICENSE.txt,META-INF/NOTICE.txt=NOTICE.txt Modified: geronimo/specs/trunk/geronimo-javamail_1.4_spec/src/main/java/javax/mail/internet/InternetHeaders.java URL: http://svn.apache.org/viewvc/geronimo/specs/trunk/geronimo-javamail_1.4_spec/src/main/java/javax/mail/internet/InternetHeaders.java?rev=688710&r1=688709&r2=688710&view=diff ============================================================================== --- geronimo/specs/trunk/geronimo-javamail_1.4_spec/src/main/java/javax/mail/internet/InternetHeaders.java (original) +++ geronimo/specs/trunk/geronimo-javamail_1.4_spec/src/main/java/javax/mail/internet/InternetHeaders.java Mon Aug 25 05:53:14 2008 @@ -535,8 +535,12 @@ // tabs and spaces are special. This is a continuation of the last header in the list. if (ch == ' ' || ch == '\t') { - InternetHeader header = (InternetHeader)headers.get(headers.size() - 1); - header.appendValue(line); + int size = headers.size(); + // it's possible that we have a leading blank line. + if (size > 0) { + InternetHeader header = (InternetHeader)headers.get(size - 1); + header.appendValue(line); + } } else { // this just gets appended to the end, preserving the addition order.