You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by ch...@apache.org on 2013/09/24 21:30:38 UTC

svn commit: r1525980 - /qpid/trunk/qpid/cpp/src/qpid/broker/Broker.cpp

Author: chug
Date: Tue Sep 24 19:30:38 2013
New Revision: 1525980

URL: http://svn.apache.org/r1525980
Log:
QPID-4604: C++ Broker queue creation limit bugfix - patch from Ernie Allen
Don't count a queue creation against a queue that already exists.


Modified:
    qpid/trunk/qpid/cpp/src/qpid/broker/Broker.cpp

Modified: qpid/trunk/qpid/cpp/src/qpid/broker/Broker.cpp
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/src/qpid/broker/Broker.cpp?rev=1525980&r1=1525979&r2=1525980&view=diff
==============================================================================
--- qpid/trunk/qpid/cpp/src/qpid/broker/Broker.cpp (original)
+++ qpid/trunk/qpid/cpp/src/qpid/broker/Broker.cpp Tue Sep 24 19:30:38 2013
@@ -1294,8 +1294,9 @@ std::pair<boost::shared_ptr<Queue>, bool
         if (!acl->authorise(userId,acl::ACT_CREATE,acl::OBJ_QUEUE,name,&params) )
             throw framing::UnauthorizedAccessException(QPID_MSG("ACL denied queue create request from " << userId));
 
-        if (!acl->approveCreateQueue(userId,name) )
-            throw framing::UnauthorizedAccessException(QPID_MSG("ACL denied queue create request from " << userId));
+        if (!queues.find(name))
+            if (!acl->approveCreateQueue(userId,name) )
+                throw framing::UnauthorizedAccessException(QPID_MSG("ACL denied queue create request from " << userId));
     }
 
     Exchange::shared_ptr alternate;



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org