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();