Repository: activemq Updated Branches: refs/heads/activemq-5.13.x 33e70292d -> 6271a7e35
https://issues.apache.org/jira/browse/AMQ-6337 Checking for the correct number of arguments before reflectively invoking a method in DefaultAuthorizationMap (cherry picked from commit 6be36305f6669ded47bcd33016583a86bdda04dd) Project: http://git-wip-us.apache.org/repos/asf/activemq/repo Commit: http://git-wip-us.apache.org/repos/asf/activemq/commit/6271a7e3 Tree: http://git-wip-us.apache.org/repos/asf/activemq/tree/6271a7e3 Diff: http://git-wip-us.apache.org/repos/asf/activemq/diff/6271a7e3 Branch: refs/heads/activemq-5.13.x Commit: 6271a7e353696e946df598af114e5f6753e16e6f Parents: 33e7029 Author: Christopher L. Shannon (cshannon) <christopher.l.shan...@gmail.com> Authored: Thu Jun 23 20:34:50 2016 +0000 Committer: Christopher L. Shannon (cshannon) <christopher.l.shan...@gmail.com> Committed: Thu Jun 23 20:36:16 2016 +0000 ---------------------------------------------------------------------- .../org/apache/activemq/security/DefaultAuthorizationMap.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/activemq/blob/6271a7e3/activemq-broker/src/main/java/org/apache/activemq/security/DefaultAuthorizationMap.java ---------------------------------------------------------------------- diff --git a/activemq-broker/src/main/java/org/apache/activemq/security/DefaultAuthorizationMap.java b/activemq-broker/src/main/java/org/apache/activemq/security/DefaultAuthorizationMap.java index 4de96f9..b249119 100644 --- a/activemq-broker/src/main/java/org/apache/activemq/security/DefaultAuthorizationMap.java +++ b/activemq-broker/src/main/java/org/apache/activemq/security/DefaultAuthorizationMap.java @@ -241,7 +241,7 @@ public class DefaultAuthorizationMap extends DestinationMap implements Authoriza Object instance; for (i = 0; i < constructors.length; i++) { Class<?>[] paramTypes = constructors[i].getParameterTypes(); - if (paramTypes.length != 0 && paramTypes[0].equals(String.class)) { + if (paramTypes.length == 1 && paramTypes[0].equals(String.class)) { break; } } @@ -253,7 +253,7 @@ public class DefaultAuthorizationMap extends DestinationMap implements Authoriza i = 0; for (i = 0; i < methods.length; i++) { Class<?>[] paramTypes = methods[i].getParameterTypes(); - if (paramTypes.length != 0 && methods[i].getName().equals("setName") && paramTypes[0].equals(String.class)) { + if (paramTypes.length == 1 && methods[i].getName().equals("setName") && paramTypes[0].equals(String.class)) { break; } }