You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commons-dev@ws.apache.org by ve...@apache.org on 2008/11/07 16:46:14 UTC
svn commit: r712169 -
/webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSSender.java
Author: veithen
Date: Fri Nov 7 07:45:47 2008
New Revision: 712169
URL: http://svn.apache.org/viewvc?rev=712169&view=rev
Log:
JMS transport: Avoid NPE if we can't get the JMSOutTransportInfo.
Modified:
webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSSender.java
Modified: webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSSender.java
URL: http://svn.apache.org/viewvc/webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSSender.java?rev=712169&r1=712168&r2=712169&view=diff
==============================================================================
--- webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSSender.java (original)
+++ webservices/commons/trunk/modules/transport/modules/jms/src/main/java/org/apache/axis2/transport/jms/JMSSender.java Fri Nov 7 07:45:47 2008
@@ -103,7 +103,7 @@
JMSConnectionFactory jmsConnectionFactory = null;
Connection connection = null; // holds a one time connection if used
- JMSOutTransportInfo jmsOut = null;
+ JMSOutTransportInfo jmsOut;
Session session = null;
Destination replyDestination = null;
@@ -146,6 +146,9 @@
session = jmsConnectionFactory.getSessionForDestination(
jmsOut.getDestination().toString());
+ } else {
+ handleException("Unable to get JMSOutTransportInfo");
+ return; // We never get here. Just make the compiler happy.
}
Destination destination = jmsOut.getDestination();