This closes #500

This commit is contained in:
Martyn Taylor 2016-05-03 14:22:32 +01:00
commit 7fb603f78f
2 changed files with 7 additions and 2 deletions

View File

@ -1151,7 +1151,10 @@ public class ConfigurationImpl implements Configuration, Serializable {
@Override
public Map<String, Set<Role>> getSecurityRoles() {
for (SecuritySettingPlugin securitySettingPlugin : securitySettingPlugins) {
securitySettings.putAll(securitySettingPlugin.getSecurityRoles());
Map<String, Set<Role>> securitySettings = securitySettingPlugin.getSecurityRoles();
if (securitySettings != null) {
securitySettings.putAll(securitySettings);
}
}
return securitySettings;
}

View File

@ -398,7 +398,9 @@ public class LegacyLDAPSecuritySettingPlugin implements SecuritySettingPlugin {
}
try {
context.close();
if (context != null) {
context.close();
}
}
catch (NamingException e) {
// ignore