[SM-963] NullPointerExceptions during JMS component initialization

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: incubation, 3.2
• Fix Version/s: 3.1.2, 3.2
• Component/s: servicemix-jms

Environment

Windows XP, JDK 1.5.0_11

• Patch Info: Patch Available
• Assignee: Guillaume Nodet
• Reporter: Piotr Bzdyl
• Created: 08/Jun/07 12:12
• Updated: 29/Jun/07 14:01
• Resolved: 29/Jun/07 14:01

Description

Currently org.apache.servicemix.jms.AbstractJmsProcessor.start() starts JMS connection (and thus starts fetching messages on all already registered and all new registered listeners) before org.apache.servicemix.jms.multiplexing.MultiplexingConsumerProcessor initializes its pendingMessages map in doStart() method.

It can be resolved by:

1) moving initialization of MultiplexingConsumerProcessor.pendingMessages before consumer.setMessageListener(this); Also initializing MultiplexingConsumerProcessor.channel field should be done before any message arrives via listener's onMessage method.

OR

2) starting JMS connection in org.apache.servicemix.jms.AbstractJmsProcessor.start() after whole initialization in subclasses is done (after doStart())

I am attaching patch for the first solution.