diff options
| author | Gordon Sim <gsim@apache.org> | 2015-02-18 23:19:12 +0000 |
|---|---|---|
| committer | Gordon Sim <gsim@apache.org> | 2015-02-18 23:19:12 +0000 |
| commit | 973758afb1efea5161d8fdf19b825b2a06ef5cff (patch) | |
| tree | 32502fa57dc2ddb8a10a6d9e1ab47bf288d3da99 /cpp | |
| parent | f7d7fa62a88124edc5af5c9c94663dd5943b8594 (diff) | |
| download | qpid-python-973758afb1efea5161d8fdf19b825b2a06ef5cff.tar.gz | |
QPID-6389: ensure source/target is null when responding to attach requests in which they are also null
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@1660763 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'cpp')
| -rw-r--r-- | cpp/src/qpid/broker/amqp/Session.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/cpp/src/qpid/broker/amqp/Session.cpp b/cpp/src/qpid/broker/amqp/Session.cpp index 538883f29a..0e44374d19 100644 --- a/cpp/src/qpid/broker/amqp/Session.cpp +++ b/cpp/src/qpid/broker/amqp/Session.cpp @@ -383,6 +383,7 @@ void Session::attach(pn_link_t* link) //i.e a subscription std::string name; if (pn_terminus_get_type(source) == PN_UNSPECIFIED) { + pn_terminus_set_type(pn_link_source(link), PN_UNSPECIFIED); throw Exception(qpid::amqp::error_conditions::PRECONDITION_FAILED, "No source specified!"); } else if (pn_terminus_is_dynamic(source)) { name = generateName(link); @@ -399,6 +400,7 @@ void Session::attach(pn_link_t* link) pn_terminus_t* target = pn_link_remote_target(link); std::string name; if (pn_terminus_get_type(target) == PN_UNSPECIFIED) { + pn_terminus_set_type(pn_link_target(link), PN_UNSPECIFIED); throw Exception(qpid::amqp::error_conditions::PRECONDITION_FAILED, "No target specified!"); } else if (pn_terminus_get_type(target) == PN_COORDINATOR) { QPID_LOG(debug, "Received attach request for incoming link to transaction coordinator on " << this); |