Skip to content

Commit bd9936d

Browse files
committed
Adds null check for user roles
1 parent 22a474a commit bd9936d

File tree

1 file changed

+9
-7
lines changed
  • components/apimgt/org.wso2.carbon.apimgt.impl/src/main/java/org/wso2/carbon/apimgt/impl/utils

1 file changed

+9
-7
lines changed

components/apimgt/org.wso2.carbon.apimgt.impl/src/main/java/org/wso2/carbon/apimgt/impl/utils/APIUtil.java

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7708,13 +7708,15 @@ private static String getActualEpPswdFromHiddenProperty(API api, Registry regist
77087708
public static boolean compareRoleList(String[] userRoleList, String accessControlRole) {
77097709
if (userRoleList != null) {
77107710
for (String userRole : userRoleList) {
7711-
if (Boolean.parseBoolean(caseSensitiveCheckEnabled)) {
7712-
if (userRole.equals(accessControlRole)) {
7713-
return true;
7714-
}
7715-
} else {
7716-
if (userRole.equalsIgnoreCase(accessControlRole)) {
7717-
return true;
7711+
if (userRole != null) {
7712+
if (Boolean.parseBoolean(caseSensitiveCheckEnabled)) {
7713+
if (userRole.equals(accessControlRole)) {
7714+
return true;
7715+
}
7716+
} else {
7717+
if (userRole.equalsIgnoreCase(accessControlRole)) {
7718+
return true;
7719+
}
77187720
}
77197721
}
77207722
}

0 commit comments

Comments
 (0)