From 6be36305f6669ded47bcd33016583a86bdda04dd Mon Sep 17 00:00:00 2001 From: "Christopher L. Shannon (cshannon)" Date: Thu, 23 Jun 2016 20:34:50 +0000 Subject: [PATCH] https://issues.apache.org/jira/browse/AMQ-6337 Checking for the correct number of arguments before reflectively invoking a method in DefaultAuthorizationMap --- .../org/apache/activemq/security/DefaultAuthorizationMap.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) 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 4de96f9f40..b249119eac 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; } }