fixed bi-dir/shutdown deadlock

1 files changed, 13 insertions, 28 deletions

diff --git a/cppe/src/IceE/Connection.cpp b/cppe/src/IceE/Connection.cpp
index 4fe759eafdd..a02a644a671 100755
--- a/cppe/src/IceE/Connection.cpp
+++ b/cppe/src/IceE/Connection.cpp
@@ -691,36 +691,21 @@ Ice::Connection::setAdapter(const ObjectAdapterPtr& adapter)
     
     assert(_state < StateClosing);
 
-    //
-    // Before we set an adapter (or reset it) we wait until the
-    // dispatch count with any old adapter is zero.
-    //
-    // A deadlock can occur if we wait while an operation is
-    // outstanding on this adapter that holds a lock while
-    // calling this function (e.g., __getDelegate).
-    //
-    // In order to avoid such a deadlock, we only wait if the new
-    // adapter is different than the current one.
-    //
-    // TODO: Verify that this fix solves all cases.
-    //
-    if(_adapter.get() != adapter.get())
-    {
-	while(_dispatchCount > 0)
-	{
-	    wait();
-	}
+    _adapter = adapter;
 
-	_adapter = adapter;
-	if(_adapter)
-	{
-	    _servantManager = _adapter->getServantManager();
-	}
-	else
-	{
-	    _servantManager = 0;
-	}
+    if(_adapter)
+    {
+	_servantManager = _adapter->getServantManager();
     }
+    else
+    {
+	_servantManager = 0;
+    }
+
+    //
+    // We never change the thread pool with which we were initially
+    // registered, even if we add or remove an object adapter.
+    //
 }
 
 ObjectAdapterPtr