diff options
author | Benoit Foucher <benoit@zeroc.com> | 2017-06-15 09:13:51 +0200 |
---|---|---|
committer | Benoit Foucher <benoit@zeroc.com> | 2017-06-15 09:13:51 +0200 |
commit | 145cf69b7cdd21e96d44c5bc5690f23e2e9d12fe (patch) | |
tree | 52444d61f215b1c20c08b0dc6141a327aeebb4e5 /java | |
parent | more BT test fixes (diff) | |
download | ice-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.java | 16 |
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; } |