From fbba7d685098eb2173db34b4e2dab4118cd47edb Mon Sep 17 00:00:00 2001 From: Michael Koch Date: Tue, 29 Apr 2003 11:36:34 +0000 Subject: PropertyPermission.java: New version from classpath 2003-04-29 Michael Koch * java/util/PropertyPermission.java: New version from classpath * java/util/ResourceBundle.java: Partly merged from classpath (getObject): Reformated. (tryBundle): Set foundBundle = null if no bundle found. From-SVN: r66214 --- libjava/java/util/PropertyPermission.java | 29 ++++++++++++++++++++++++++--- 1 file changed, 26 insertions(+), 3 deletions(-) (limited to 'libjava/java/util/PropertyPermission.java') diff --git a/libjava/java/util/PropertyPermission.java b/libjava/java/util/PropertyPermission.java index bb03e45f362..0d439d888a7 100644 --- a/libjava/java/util/PropertyPermission.java +++ b/libjava/java/util/PropertyPermission.java @@ -121,7 +121,7 @@ public final class PropertyPermission extends BasicPermission super(name); if (actions == null) throw new IllegalArgumentException(); - setActions(actions.toLowerCase()); + setActions(actions); } /** @@ -134,14 +134,37 @@ public final class PropertyPermission extends BasicPermission */ private void setActions(String str) { + // Initialising the class java.util.Locale ... + // tries to initialise the Locale.defaultLocale static + // which calls System.getProperty, + // which calls SecurityManager.checkPropertiesAccess, + // which creates a PropertyPermission with action "read,write", + // which calls setActions("read,write"). + // If we now were to call toLowerCase on 'str', + // this would call Locale.getDefault() which returns null + // because Locale.defaultLocale hasn't been set yet + // then toLowerCase will fail with a null pointer exception. + // + // The solution is to take a punt on 'str' being lower case, and + // test accordingly. If that fails, we convert 'str' to lower case + // and try the tests again. if ("read".equals(str)) actions = READ; else if ("write".equals(str)) actions = WRITE; else if ("read,write".equals(str) || "write,read".equals(str)) actions = READ | WRITE; - else - throw new IllegalArgumentException("illegal action " + str); + else { + String lstr = str.toLowerCase(); + if ("read".equals(lstr)) + actions = READ; + else if ("write".equals(lstr)) + actions = WRITE; + else if ("read,write".equals(lstr) || "write,read".equals(lstr)) + actions = READ | WRITE; + else + throw new IllegalArgumentException("illegal action " + str); + } } /** -- cgit v1.2.3