From 0d6d2adbb434f05f63414fc3fb5441345666a862 Mon Sep 17 00:00:00 2001 From: Robert Godfrey Date: Mon, 3 Mar 2014 15:35:25 +0000 Subject: QPID-5596 : [Java] Address bugs found by FindBugs git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1573586 13f79535-47bb-0310-9956-ffa450edef68 --- .../apache/qpid/server/security/access/config/PlainConfiguration.java | 2 +- .../java/org/apache/qpid/server/security/access/config/RuleSet.java | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) (limited to 'qpid/java/broker-plugins/access-control/src') diff --git a/qpid/java/broker-plugins/access-control/src/main/java/org/apache/qpid/server/security/access/config/PlainConfiguration.java b/qpid/java/broker-plugins/access-control/src/main/java/org/apache/qpid/server/security/access/config/PlainConfiguration.java index 31c345f718..34585bb4f6 100644 --- a/qpid/java/broker-plugins/access-control/src/main/java/org/apache/qpid/server/security/access/config/PlainConfiguration.java +++ b/qpid/java/broker-plugins/access-control/src/main/java/org/apache/qpid/server/security/access/config/PlainConfiguration.java @@ -148,7 +148,7 @@ public class PlainConfiguration extends AbstractConfiguration { if(StringUtils.equalsIgnoreCase("GROUP", first)) { - throw new IllegalConfigurationException(String.format("GROUP keyword not supported. Groups should defined via a Group Provider, not in the ACL file.", getLine())); + throw new IllegalConfigurationException(String.format("GROUP keyword not supported at line %d. Groups should defined via a Group Provider, not in the ACL file.", getLine())); } else if (StringUtils.equalsIgnoreCase(CONFIG, first)) { diff --git a/qpid/java/broker-plugins/access-control/src/main/java/org/apache/qpid/server/security/access/config/RuleSet.java b/qpid/java/broker-plugins/access-control/src/main/java/org/apache/qpid/server/security/access/config/RuleSet.java index 2e2b6b2041..fcb5bcbf70 100644 --- a/qpid/java/broker-plugins/access-control/src/main/java/org/apache/qpid/server/security/access/config/RuleSet.java +++ b/qpid/java/broker-plugins/access-control/src/main/java/org/apache/qpid/server/security/access/config/RuleSet.java @@ -185,7 +185,6 @@ public class RuleSet public void addRule(Integer number, String identity, Permission permission, AclAction action) { - _cache.clear(); if (!action.isAllowed()) { @@ -211,7 +210,7 @@ public class RuleSet } // save rule - _cache.remove(identity); + _cache.clear(); _rules.put(rule.getNumber(), rule); } -- cgit v1.2.1