[jboss-remoting-commits] JBoss Remoting SVN: r3697 - remoting2/branches/2.x/src/main/org/jboss/remoting.

[jboss-remoting-commits at lists.jboss.org]

Author: ron.sigal at jboss.com
Date: 2008-03-20 21:30:05 -0400 (Thu, 20 Mar 2008)
New Revision: 3697

Modified:
   remoting2/branches/2.x/src/main/org/jboss/remoting/InvokerLocator.java
Log:
JBREM-934: Put System.getProperty() in AccessController.doPrivileged() call.

Modified: remoting2/branches/2.x/src/main/org/jboss/remoting/InvokerLocator.java
===================================================================
--- remoting2/branches/2.x/src/main/org/jboss/remoting/InvokerLocator.java	2008-03-21 01:23:14 UTC (rev 3696)
+++ remoting2/branches/2.x/src/main/org/jboss/remoting/InvokerLocator.java	2008-03-21 01:30:05 UTC (rev 3697)
@@ -28,6 +28,8 @@
 import java.net.URI;
 import java.net.URISyntaxException;
 import java.net.UnknownHostException;
+import java.security.AccessController;
+import java.security.PrivilegedAction;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Iterator;
@@ -306,10 +308,21 @@
    {
       boolean doLegacyParsing = false;
       if (legacyParsingFlag != null)
+      {
          doLegacyParsing = legacyParsingFlag.booleanValue();
-      else if (System.getProperty(LEGACY_PARSING) != null)
-         doLegacyParsing = Boolean.getBoolean(LEGACY_PARSING);
-      
+      }
+      else 
+      {
+         String s = (String)AccessController.doPrivileged( new PrivilegedAction()
+         {
+            public Object run()
+            {
+               return System.getProperty(LEGACY_PARSING);
+            }
+         });
+         doLegacyParsing = "true".equalsIgnoreCase(s);
+      }
+
       if (doLegacyParsing)
       {
          log.warn("using deprecated legacy URL parsing routine");