diff options
| author | Gordon Sim <gsim@apache.org> | 2012-04-23 20:58:00 +0000 |
|---|---|---|
| committer | Gordon Sim <gsim@apache.org> | 2012-04-23 20:58:00 +0000 |
| commit | 53a875d0fa717f5a58cf76bdc2faef638721d401 (patch) | |
| tree | 18dd73a121eb97b20ccb4f26b065626ad4e1a095 /qpid/cpp/src | |
| parent | 813582c0cf24fda9481d0a666be41a9e78b27452 (diff) | |
| download | qpid-python-53a875d0fa717f5a58cf76bdc2faef638721d401.tar.gz | |
QPID-3970: Check alternate exchange actually exists before proceeding with reroute request
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1329438 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'qpid/cpp/src')
| -rw-r--r-- | qpid/cpp/src/qpid/broker/Queue.cpp | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/qpid/cpp/src/qpid/broker/Queue.cpp b/qpid/cpp/src/qpid/broker/Queue.cpp index 4f114fa0a7..e7305c021d 100644 --- a/qpid/cpp/src/qpid/broker/Queue.cpp +++ b/qpid/cpp/src/qpid/broker/Queue.cpp @@ -1625,9 +1625,14 @@ Manageable::status_t Queue::ManagementMethod (uint32_t methodId, Args& args, str { _qmf::ArgsQueueReroute& rerouteArgs = (_qmf::ArgsQueueReroute&) args; boost::shared_ptr<Exchange> dest; - if (rerouteArgs.i_useAltExchange) + if (rerouteArgs.i_useAltExchange) { + if (!alternateExchange) { + status = Manageable::STATUS_PARAMETER_INVALID; + etext = "No alternate-exchange defined"; + break; + } dest = alternateExchange; - else { + } else { try { dest = broker->getExchanges().get(rerouteArgs.i_exchange); } catch(const std::exception&) { |