QPID-856 : Moved unbind out of the synchronized block as it doesn't need to be done there.

git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/branches/M2.1@637828 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 7 insertions, 2 deletions

diff --git a/java/client/src/main/java/org/apache/qpid/client/transport/TransportConnection.java b/java/client/src/main/java/org/apache/qpid/client/transport/TransportConnection.java
index 78b54a8de9..2a5365c322 100644
--- a/java/client/src/main/java/org/apache/qpid/client/transport/TransportConnection.java
+++ b/java/client/src/main/java/org/apache/qpid/client/transport/TransportConnection.java
@@ -332,16 +332,21 @@ public class TransportConnection
 
     public static void killVMBroker(int port)
     {
+        VmPipeAddress pipe;
         synchronized (_inVmPipeAddress)
         {
-            VmPipeAddress pipe = (VmPipeAddress) _inVmPipeAddress.get(port);
+            pipe = (VmPipeAddress) _inVmPipeAddress.get(port);
             if (pipe != null)
             {
                 _logger.info("Killing VM Broker:" + port);
                 _inVmPipeAddress.remove(port);
-                _acceptor.unbind(pipe);
             }
         }
+        //This doesn't need to be sychronized
+        if (pipe != null)
+        {
+            _acceptor.unbind(pipe);
+        }
     }
 
 }