From 973758afb1efea5161d8fdf19b825b2a06ef5cff Mon Sep 17 00:00:00 2001 From: Gordon Sim Date: Wed, 18 Feb 2015 23:19:12 +0000 Subject: 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 --- cpp/src/qpid/broker/amqp/Session.cpp | 2 ++ 1 file changed, 2 insertions(+) (limited to 'cpp') 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); -- cgit v1.2.1