tidy up fix (exception was being lost) for URL: http://svn.apache.org…

…/viewvc?rev=1209841&view=rev Only stop transport for java.lang.SecurityException - as some tests rely on Connection continung after an exception from the broker related to changes in support of fix for https://issues.apache.org/jira/browse/AMQ-3294 and https://issues.apache.org/jira/browse/AMQ-1928 git-svn-id: https://svn.apache.org/repos/asf/activemq/trunk@1211844 13f79535-47bb-0310-9956-ffa450edef68
apache · Dec 8, 2011 · 9df9d3e · 9df9d3e
1 parent 760c629
commit 9df9d3e
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/activemq-core/src/main/java/org/apache/activemq/ActiveMQConnection.java b/activemq-core/src/main/java/org/apache/activemq/ActiveMQConnection.java
@@ -1303,8 +1303,8 @@ public Response syncSendPacket(Command command) throws JMSException {
                         }
                         JMSException jmsEx = null;
                         try {
-                         jmsEx = JMSExceptionSupport.create(er.getException());
-                        }catch(Throwable e) {
+                            jmsEx = JMSExceptionSupport.create(er.getException());
+                        } catch(Throwable e) {
                             LOG.error("Caught an exception trying to create a JMSException for " +er.getException(),e);
                         }
                         //dispose of transport for security exceptions
@@ -1313,9 +1313,9 @@ public Response syncSendPacket(Command command) throws JMSException {
                             if (null != t){
                                 ServiceSupport.dispose(t);
                             }
-                            if(jmsEx !=null) {
-                                throw jmsEx;
-                            }
+                        }
+                        if (jmsEx !=null) {
+                            throw jmsEx;
                         }
                     }
                 }