diff options
| author | Gordon Sim <gsim@apache.org> | 2007-09-17 14:08:17 +0000 |
|---|---|---|
| committer | Gordon Sim <gsim@apache.org> | 2007-09-17 14:08:17 +0000 |
| commit | 66b1f5db27110f82d530846c450dc1b6965de7f1 (patch) | |
| tree | f271bff2ec52bfd1749475e837a7dd0a2aa8801e /cpp/src/qpid/broker/Session.cpp | |
| parent | c2fb7475ccffba8ab0d214dfbb65e132788186d7 (diff) | |
| download | qpid-python-66b1f5db27110f82d530846c450dc1b6965de7f1.tar.gz | |
Fix for race between flush and delivery.
git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/trunk/qpid@576448 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'cpp/src/qpid/broker/Session.cpp')
| -rw-r--r-- | cpp/src/qpid/broker/Session.cpp | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/cpp/src/qpid/broker/Session.cpp b/cpp/src/qpid/broker/Session.cpp index d3f82655d0..8e8632e922 100644 --- a/cpp/src/qpid/broker/Session.cpp +++ b/cpp/src/qpid/broker/Session.cpp @@ -535,7 +535,13 @@ void Session::ConsumerImpl::addMessageCredit(uint32_t value) void Session::ConsumerImpl::flush() { + //need to prevent delivery after requestDispatch returns but + //before credit is reduced to zero; TODO: come up with better + //implementation of flush. + Mutex::ScopedLock l(lock); queue->requestDispatch(this, true); + byteCredit = 0; + msgCredit = 0; } void Session::ConsumerImpl::stop() |