diff options
| author | Gordon Sim <gsim@apache.org> | 2007-10-17 13:45:27 +0000 |
|---|---|---|
| committer | Gordon Sim <gsim@apache.org> | 2007-10-17 13:45:27 +0000 |
| commit | 2b382c2ab784f310d0ed36825668a6368f22a668 (patch) | |
| tree | 3dc733e109354142fd60350b9da724401bcd5174 /qpid/cpp/src/tests/HeadersExchangeTest.cpp | |
| parent | 3bd42ed62494d189cee70087ead1794a4e685a2a (diff) | |
| download | qpid-python-2b382c2ab784f310d0ed36825668a6368f22a668.tar.gz | |
Fix to headers exchanges bind: need to check the match value is present before dereferencing
Added tests for this.
git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/trunk@585503 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'qpid/cpp/src/tests/HeadersExchangeTest.cpp')
| -rw-r--r-- | qpid/cpp/src/tests/HeadersExchangeTest.cpp | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/qpid/cpp/src/tests/HeadersExchangeTest.cpp b/qpid/cpp/src/tests/HeadersExchangeTest.cpp index 14c60449ec..c47266caa6 100644 --- a/qpid/cpp/src/tests/HeadersExchangeTest.cpp +++ b/qpid/cpp/src/tests/HeadersExchangeTest.cpp @@ -19,6 +19,7 @@ * */ +#include "qpid/QpidError.h" #include "qpid/broker/HeadersExchange.h" #include "qpid/framing/FieldTable.h" #include "qpid/framing/FieldValue.h" @@ -35,6 +36,7 @@ class HeadersExchangeTest : public CppUnit::TestCase CPPUNIT_TEST(testMatchEmptyValue); CPPUNIT_TEST(testMatchEmptyArgs); CPPUNIT_TEST(testMatchNoXMatch); + CPPUNIT_TEST(testBindNoXMatch); CPPUNIT_TEST_SUITE_END(); public: @@ -107,6 +109,20 @@ class HeadersExchangeTest : public CppUnit::TestCase CPPUNIT_ASSERT(!HeadersExchange::match(b, m)); } + void testBindNoXMatch() + { + HeadersExchange exchange("test"); + Queue::shared_ptr queue; + std::string key; + FieldTable args; + try { + //just checking this doesn't cause assertion etc + exchange.bind(queue, key, &args); + } catch(qpid::QpidError&) { + //expected + } + } + }; |