MailListenerChainService.processMail doesn't handle null message properly. -------------------------------------------------------------------------- Key: JBMAIL-241 URL: http://jira.jboss.com/jira/browse/JBMAIL-241 Project: JBoss Mail ** Closed - moved to http://buni.org ** Issue Type: Patch Security Level: Public(Everyone can see) Affects Versions: 1.0-M2, 1.0-M3, 1.0-M4, 1.0-M5, 1.0-RC1 Environment: All Reporter: David Fuelling Original Estimate: 15 minutes Remaining Estimate: 15 minutes If a given mail listner (i.e., JMSMailListener) completely processes a mail, the function contract stipulates that the listener should return 'null'. Unfortunately, not all of the JBCS listeners deal well with a null message, so the MailListenerChainService should detect if a null message is received from any listener. If so, the listener should abort the processing of the remaining listeners. The Patch is: Index: MailListenerChainService.java =================================================================== RCS file: /cvsroot/jboss/jboss-mail/src/java/org/jboss/mail/MailListenerChainService.java,v retrieving revision 1.4 diff -u -r1.4 MailListenerChainService.java --- MailListenerChainService.java 31 Mar 2006 22:34:51 -0000 1.4 +++ MailListenerChainService.java 7 Jul 2006 15:03:02 -0000 @@ -187,6 +187,8 @@ for (int i = 0; i < listeners.size(); i++) { MailListener listener = ((PositionedListener) listeners.get(i)).listener; msg = listener.send(msg); + if(msg == null) + break; } return msg; }