summaryrefslogtreecommitdiff
path: root/java
diff options
context:
space:
mode:
authorBenoit Foucher <benoit@zeroc.com>2017-06-15 09:13:51 +0200
committerBenoit Foucher <benoit@zeroc.com>2017-06-15 09:13:51 +0200
commit145cf69b7cdd21e96d44c5bc5690f23e2e9d12fe (patch)
tree52444d61f215b1c20c08b0dc6141a327aeebb4e5 /java
parentmore BT test fixes (diff)
downloadice-145cf69b7cdd21e96d44c5bc5690f23e2e9d12fe.tar.bz2
ice-145cf69b7cdd21e96d44c5bc5690f23e2e9d12fe.tar.xz
ice-145cf69b7cdd21e96d44c5bc5690f23e2e9d12fe.zip
Fixed ICE-8056 - communicator waitForShutdown hang
Diffstat (limited to 'java')
-rw-r--r--java/src/Ice/src/main/java/com/zeroc/IceInternal/FactoryACMMonitor.java16
1 files changed, 15 insertions, 1 deletions
diff --git a/java/src/Ice/src/main/java/com/zeroc/IceInternal/FactoryACMMonitor.java b/java/src/Ice/src/main/java/com/zeroc/IceInternal/FactoryACMMonitor.java
index 46bf10b1733..c5ca7874b05 100644
--- a/java/src/Ice/src/main/java/com/zeroc/IceInternal/FactoryACMMonitor.java
+++ b/java/src/Ice/src/main/java/com/zeroc/IceInternal/FactoryACMMonitor.java
@@ -55,6 +55,20 @@ class FactoryACMMonitor implements ACMMonitor
{
if(_instance == null)
{
+ //
+ // Ensure all the connections have been cleared, it's important to wait here
+ // to prevent the timer destruction in IceInternal::Instance::destroy.
+ //
+ while(!_connections.isEmpty())
+ {
+ try
+ {
+ wait();
+ }
+ catch(InterruptedException ex)
+ {
+ }
+ }
return;
}
@@ -194,7 +208,7 @@ class FactoryACMMonitor implements ACMMonitor
if(_instance == null)
{
_connections.clear();
- notify();
+ notifyAll();
return;
}