From 604582905ab607be1fa5b43130426bb31faaad55 Mon Sep 17 00:00:00 2001 From: Alan Conway Date: Wed, 18 Jan 2012 22:08:45 +0000 Subject: QPID-3603: Act on bridge cancellations before creations This ensures that recreation of a bridge does not result in two active subscriptions (evench can cause confusion for replicators, even if they are both active only for a very short period). git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@1233086 13f79535-47bb-0310-9956-ffa450edef68 --- cpp/src/qpid/broker/Link.cpp | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) (limited to 'cpp/src') diff --git a/cpp/src/qpid/broker/Link.cpp b/cpp/src/qpid/broker/Link.cpp index 7e1767fed7..a27cb7c5ac 100644 --- a/cpp/src/qpid/broker/Link.cpp +++ b/cpp/src/qpid/broker/Link.cpp @@ -261,7 +261,15 @@ void Link::ioThreadProcessing() active.erase(removed, active.end()); } - //process any pending creates and/or cancellations + //process any pending creates and/or cancellations (do + //cancellations first in case any of the creates represent + //recreation of cancelled subscriptions + if (!cancellations.empty()) { + for (Bridges::iterator i = cancellations.begin(); i != cancellations.end(); ++i) { + (*i)->cancel(*connection); + } + cancellations.clear(); + } if (!created.empty()) { for (Bridges::iterator i = created.begin(); i != created.end(); ++i) { active.push_back(*i); @@ -269,12 +277,6 @@ void Link::ioThreadProcessing() } created.clear(); } - if (!cancellations.empty()) { - for (Bridges::iterator i = cancellations.begin(); i != cancellations.end(); ++i) { - (*i)->cancel(*connection); - } - cancellations.clear(); - } } void Link::setConnection(Connection* c) -- cgit v1.2.1