You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by rg...@apache.org on 2015/10/16 15:27:03 UTC

svn commit: r1708992 - /qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/security/SecurityManager.java

Author: rgodfrey
Date: Fri Oct 16 13:27:02 2015
New Revision: 1708992

URL: http://svn.apache.org/viewvc?rev=1708992&view=rev
Log:
QPID-6781 : Avoid wasteful calls to Subject.getSubject() when the subject is known

Modified:
    qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/security/SecurityManager.java

Modified: qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/security/SecurityManager.java
URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/security/SecurityManager.java?rev=1708992&r1=1708991&r2=1708992&view=diff
==============================================================================
--- qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/security/SecurityManager.java (original)
+++ qpid/java/trunk/broker-core/src/main/java/org/apache/qpid/server/security/SecurityManager.java Fri Oct 16 13:27:02 2015
@@ -667,7 +667,7 @@ public class SecurityManager
             check = new PublishAccessCheck(new ObjectProperties(virtualHostName, exchangeName, routingKey, immediate));
             _publishAccessCheckCache.putIfAbsent(key, check);
         }
-        if(!checkAllPlugins(check))
+        if(!checkAllPlugins(check, currentSubject))
         {
             throw new AccessControlException("Permission denied, publish to: exchange-name '" + exchangeName + "'");
         }



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