Apache
/
activemq
mirror of https://github.com/apache/activemq.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

https://issues.apache.org/jira/browse/AMQ-6337 

Checking for the correct number of arguments before reflectively
invoking a method in DefaultAuthorizationMap
This commit is contained in:
Christopher L. Shannon (cshannon) 2016-06-23 20:34:50 +00:00
parent 3dfda807f6
commit 6be36305f6
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
activemq-broker/src/main/java/org/apache/activemq/security/DefaultAuthorizationMap.java
View File

@ -241,7 +241,7 @@ public class DefaultAuthorizationMap extends DestinationMap implements Authoriza
Object instance; Object instance;
for (i = 0; i < constructors.length; i++) { for (i = 0; i < constructors.length; i++) {
Class<?>[] paramTypes = constructors[i].getParameterTypes(); Class<?>[] paramTypes = constructors[i].getParameterTypes();
if (paramTypes.length != 0 && paramTypes[0].equals(String.class)) { if (paramTypes.length == 1 && paramTypes[0].equals(String.class)) {
break; break;
} }
} }
@ -253,7 +253,7 @@ public class DefaultAuthorizationMap extends DestinationMap implements Authoriza
i = 0; i = 0;
for (i = 0; i < methods.length; i++) { for (i = 0; i < methods.length; i++) {
Class<?>[] paramTypes = methods[i].getParameterTypes(); 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; break;
} }
} }