From e8578e6352e2ea6227d0abcf4f6f35694da9d4bd Mon Sep 17 00:00:00 2001 From: Sebastian Sdorra Date: Fri, 18 Feb 2011 11:46:09 +0100 Subject: [PATCH] fix NullPointerException --- .../scm/web/security/BasicSecurityContext.java | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/scm-webapp/src/main/java/sonia/scm/web/security/BasicSecurityContext.java b/scm-webapp/src/main/java/sonia/scm/web/security/BasicSecurityContext.java index b14674de93..943cccc982 100644 --- a/scm-webapp/src/main/java/sonia/scm/web/security/BasicSecurityContext.java +++ b/scm-webapp/src/main/java/sonia/scm/web/security/BasicSecurityContext.java @@ -298,11 +298,22 @@ public class BasicSecurityContext implements WebSecurityContext */ private boolean isAdmin() { - boolean result = configuration.getAdminUsers().contains(user.getName()); + boolean result = false; + Set adminUsers = configuration.getAdminUsers(); + + if (adminUsers != null) + { + result = adminUsers.contains(user.getName()); + } if (!result) { - result = Util.containsOne(configuration.getAdminGroups(), groups); + Set adminGroups = configuration.getAdminGroups(); + + if (adminGroups != null) + { + result = Util.containsOne(adminGroups, groups); + } } return result;