]
David Fuelling updated JBMAIL-247:
----------------------------------
Issue Type: Patch (was: Bug)
MessageData NPE in getBoundary when Content-Type header is missing
------------------------------------------------------------------
Key: JBMAIL-247
URL:
http://jira.jboss.com/jira/browse/JBMAIL-247
Project: JBoss Mail
Issue Type: Patch
Security Level: Public(Everyone can see)
Components: IMAP, Mail Server APIs, POP, WebMail
Affects Versions: 1.5-M4, 1.5-M5
Environment: WinXP, JBossAS 4.0.4GA_Patch1, JBCS Head as of July 31st, 2006.
Reporter: David Fuelling
Assigned To: Andrew Oliver
Priority: Minor
Original Estimate: 5 minutes
Remaining Estimate: 5 minutes
See forum post here:
http://www.jboss.org/index.html?module=bb&op=viewtopic&t=88893
Here is a patch. It simply moves the creation of the "header" string array
inside of the try/catch block. If an exception is thrown, it is caught and ignored.
Index: MessageData.java
===================================================================
RCS file: /cvsroot/jboss/jboss-mail/src/java/org/jboss/mail/mailbox/MessageData.java,v
retrieving revision 1.21
diff -u -r1.21 MessageData.java
--- MessageData.java 11 Jun 2006 14:06:23 -0000 1.21
+++ MessageData.java 18 Aug 2006 15:16:02 -0000
@@ -342,9 +342,9 @@
}
public String getBoundary() {
- String[] header = getHeader("Content-Type").split("\\r\\n");
- String retval = null;
+ String retval = null;
try {
+ String[] header = getHeader("Content-Type").split("\\r\\n");
for (int i = 0; header != null && i < header.length; i++) {
if( header[i].indexOf("boundary=\"") >-1) {
String[] temp = header[i].split("boundary\\=\\\"");
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: