keycloak-aplcache

Merge pull request #4674 from thomasdarimont/issue/fix-npe-in-userpermissions KEYCLOAK-5841 …

11/15/2017 10:22:44 AM

Fix NPE in deletePermissionSetup in UserPermissions

Pedro Igor

Commit: 1bd2f0e

Tree: da360ce

Parents: 393fae7
a5b73a3

Changes

services/src/main/java/org/keycloak/services/resources/admin/permissions/UserPermissions.java 2(+1 -1)

Details

services/src/main/java/org/keycloak/services/resources/admin/permissions/UserPermissions.java 2(+1 -1)

diff --git a/services/src/main/java/org/keycloak/services/resources/admin/permissions/UserPermissions.java b/services/src/main/java/org/keycloak/services/resources/admin/permissions/UserPermissions.java
index 55b7fb4..a8d9c8b 100644
--- a/services/src/main/java/org/keycloak/services/resources/admin/permissions/UserPermissions.java
+++ b/services/src/main/java/org/keycloak/services/resources/admin/permissions/UserPermissions.java
@@ -195,7 +195,7 @@ class UserPermissions implements UserPermissionEvaluator, UserPermissionManageme
 
         }
         Resource usersResource = authz.getStoreFactory().getResourceStore().findByName(USERS_RESOURCE, server.getId());
-        if (usersResource == null) {
+        if (usersResource != null) {
             authz.getStoreFactory().getResourceStore().delete(usersResource.getId());
         }
     }