diff options
author | Benoit Foucher <benoit@zeroc.com> | 2020-02-20 12:45:29 +0100 |
---|---|---|
committer | Benoit Foucher <benoit@zeroc.com> | 2020-02-20 12:45:29 +0100 |
commit | d5b4c4091ce634d8e99ab0150acd853bd209df2a (patch) | |
tree | c4b415bf56f65422b4ab0efd8e8602647af66e99 /cpp/src | |
parent | Added comments to IceDiscovery.ice fixes #697 (diff) | |
download | ice-d5b4c4091ce634d8e99ab0150acd853bd209df2a.tar.bz2 ice-d5b4c4091ce634d8e99ab0150acd853bd209df2a.tar.xz ice-d5b4c4091ce634d8e99ab0150acd853bd209df2a.zip |
Fixed invalid access to _instance in RetryQueue, fixes #681
Diffstat (limited to 'cpp/src')
-rw-r--r-- | cpp/src/Ice/RetryQueue.cpp | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/cpp/src/Ice/RetryQueue.cpp b/cpp/src/Ice/RetryQueue.cpp index 8f9694416fd..b52a6b59fc9 100644 --- a/cpp/src/Ice/RetryQueue.cpp +++ b/cpp/src/Ice/RetryQueue.cpp @@ -144,11 +144,14 @@ IceInternal::RetryQueue::cancel(const RetryTaskPtr& task) Lock sync(*this); if(_requests.erase(task) > 0) { - if(!_instance && _requests.empty()) + if(_instance) + { + return _instance->timer()->cancel(task); + } + else if(_requests.empty()) { notify(); // If we are destroying the queue, destroy is probably waiting on the queue to be empty. } - return _instance->timer()->cancel(task); } return false; } |