summaryrefslogtreecommitdiff
path: root/cpp/src
diff options
context:
space:
mode:
authorBenoit Foucher <benoit@zeroc.com>2020-02-20 12:45:29 +0100
committerBenoit Foucher <benoit@zeroc.com>2020-02-20 12:45:29 +0100
commitd5b4c4091ce634d8e99ab0150acd853bd209df2a (patch)
treec4b415bf56f65422b4ab0efd8e8602647af66e99 /cpp/src
parentAdded comments to IceDiscovery.ice fixes #697 (diff)
downloadice-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.cpp7
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;
}