Repository: shiro Updated Branches: refs/heads/1.2.x 2a9e56439 -> 7e4c1f014
SHIRO-562: Avoid calling trim() twice Project: http://git-wip-us.apache.org/repos/asf/shiro/repo Commit: http://git-wip-us.apache.org/repos/asf/shiro/commit/7e4c1f01 Tree: http://git-wip-us.apache.org/repos/asf/shiro/tree/7e4c1f01 Diff: http://git-wip-us.apache.org/repos/asf/shiro/diff/7e4c1f01 Branch: refs/heads/1.2.x Commit: 7e4c1f0140a040ef85ded0e3476320517c5d0dd0 Parents: 2a9e564 Author: Andreas Kohn <[email protected]> Authored: Fri Mar 18 14:17:48 2016 +0100 Committer: bdemers <[email protected]> Committed: Mon Mar 21 12:48:41 2016 -0400 ---------------------------------------------------------------------- .../org/apache/shiro/authz/permission/WildcardPermission.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/shiro/blob/7e4c1f01/core/src/main/java/org/apache/shiro/authz/permission/WildcardPermission.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/org/apache/shiro/authz/permission/WildcardPermission.java b/core/src/main/java/org/apache/shiro/authz/permission/WildcardPermission.java index 1a755ee..cb9d14e 100644 --- a/core/src/main/java/org/apache/shiro/authz/permission/WildcardPermission.java +++ b/core/src/main/java/org/apache/shiro/authz/permission/WildcardPermission.java @@ -20,6 +20,7 @@ package org.apache.shiro.authz.permission; import org.apache.shiro.authz.Permission; import org.apache.shiro.util.CollectionUtils; +import org.apache.shiro.util.StringUtils; import java.io.Serializable; import java.util.ArrayList; @@ -147,12 +148,12 @@ public class WildcardPermission implements Permission, Serializable { } protected void setParts(String wildcardString, boolean caseSensitive) { - if (wildcardString == null || wildcardString.trim().length() == 0) { + wildcardString = StringUtils.clean(wildcardString); + + if (wildcardString == null || wildcardString.isEmpty()) { throw new IllegalArgumentException("Wildcard string cannot be null or empty. Make sure permission strings are properly formatted."); } - wildcardString = wildcardString.trim(); - List<String> parts = CollectionUtils.asList(wildcardString.split(PART_DIVIDER_TOKEN)); this.parts = new ArrayList<Set<String>>();
