From 7db3e48be0df04c7b315d0db56f1bb7f602f4f8a Mon Sep 17 00:00:00 2001
From: Robert Godfrey <rgodfrey@apache.org>
Date: Fri, 10 Aug 2007 13:52:48 +0000
Subject: QPID-547 : Stop session closure while in message dispatch

git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/branches/M2@564593 13f79535-47bb-0310-9956-ffa450edef68
---
 .../src/main/java/org/apache/qpid/client/AMQSession.java    | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

(limited to 'java/client/src')

diff --git a/java/client/src/main/java/org/apache/qpid/client/AMQSession.java b/java/client/src/main/java/org/apache/qpid/client/AMQSession.java
index 99bf085aa6..4e259f651c 100644
--- a/java/client/src/main/java/org/apache/qpid/client/AMQSession.java
+++ b/java/client/src/main/java/org/apache/qpid/client/AMQSession.java
@@ -291,6 +291,7 @@ public class AMQSession extends Closeable implements Session, QueueSession, Topi
 
     /** Indicates that runtime exceptions should be generated on vilations of the strict AMQP. */
     private final boolean _strictAMQPFATAL;
+    private final Object _messageDeliveryLock = new Object();
 
     /**
      * Creates a new session on a connection.
@@ -512,6 +513,9 @@ public class AMQSession extends Closeable implements Session, QueueSession, Topi
                 + Arrays.asList(Thread.currentThread().getStackTrace()).subList(3, 6));
         }
 
+        synchronized(_messageDeliveryLock)
+        {
+
         // We must close down all producers and consumers in an orderly fashion. This is the only method
         // that can be called from a different thread of control from the one controlling the session.
         synchronized (_connection.getFailoverMutex())
@@ -558,6 +562,7 @@ public class AMQSession extends Closeable implements Session, QueueSession, Topi
                 }
             }
         }
+        }
     }
 
     /**
@@ -567,6 +572,8 @@ public class AMQSession extends Closeable implements Session, QueueSession, Topi
      */
     public void closed(Throwable e) throws JMSException
     {
+        synchronized(_messageDeliveryLock)
+        {
         synchronized (_connection.getFailoverMutex())
         {
             // An AMQException has an error code and message already and will be passed in when closure occurs as a
@@ -585,6 +592,7 @@ public class AMQSession extends Closeable implements Session, QueueSession, Topi
             _connection.deregisterSession(_channelId);
             closeProducersAndConsumers(amqe);
         }
+        }
     }
 
     /**
@@ -2662,7 +2670,10 @@ public class AMQSession extends Closeable implements Session, QueueSession, Topi
                             _lock.wait();
                         }
 
-                        dispatchMessage(message);
+                        synchronized(_messageDeliveryLock)
+                        {
+                            dispatchMessage(message);
+                        }
 
                         while (connectionStopped())
                         {
-- 
cgit v1.2.1