diff options
| author | Alan Conway <aconway@apache.org> | 2010-03-15 20:08:10 +0000 |
|---|---|---|
| committer | Alan Conway <aconway@apache.org> | 2010-03-15 20:08:10 +0000 |
| commit | 600a7bfe6b25448f68f266996a2f8e6700c69eac (patch) | |
| tree | 5a45a4ef67cc2ff603635028e7d381429089272d /qpid/cpp/src/tests | |
| parent | c698c596d5d79dcaee85227186d0361ff76a0dae (diff) | |
| download | qpid-python-600a7bfe6b25448f68f266996a2f8e6700c69eac.tar.gz | |
Work-around for race condition in the IO layer.
Added a 100ms sleep in UpdateClient between calling
Connection::close() and destroying the Connection object.
This appears to work around the race condition described in
https://bugzilla.redhat.com/show_bug.cgi?id=568831.
which was causing brokers to seg fault sporadically at the end of
giving an update.
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@923414 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'qpid/cpp/src/tests')
| -rw-r--r-- | qpid/cpp/src/tests/ClusterFixture.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/qpid/cpp/src/tests/ClusterFixture.cpp b/qpid/cpp/src/tests/ClusterFixture.cpp index fd90ed170e..b7e8e88abf 100644 --- a/qpid/cpp/src/tests/ClusterFixture.cpp +++ b/qpid/cpp/src/tests/ClusterFixture.cpp @@ -130,11 +130,11 @@ void ClusterFixture::kill(size_t n, int sig) { forkedBrokers[n]->kill(sig); } -/** Kill a broker and suppressing errors from closing connection c. */ +/** Kill a broker and suppress errors from closing connection c. */ void ClusterFixture::killWithSilencer(size_t n, client::Connection& c, int sig) { ScopedSuppressLogging sl; - kill(n,sig); try { c.close(); } catch(...) {} + kill(n,sig); } /** |