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

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

Author: ron.sigal at jboss.com
Date: 2008-04-02 00:25:01 -0400 (Wed, 02 Apr 2008)
New Revision: 3852

Modified:
   remoting2/branches/2.x/src/main/org/jboss/remoting/transport/rmi/RemotingRMIClientSocketFactory.java
Log:
JBREM-934: Put InetAddress.getByName() call in AccessController.doPrivileged() call.

Modified: remoting2/branches/2.x/src/main/org/jboss/remoting/transport/rmi/RemotingRMIClientSocketFactory.java
===================================================================
--- remoting2/branches/2.x/src/main/org/jboss/remoting/transport/rmi/RemotingRMIClientSocketFactory.java	2008-04-02 04:23:27 UTC (rev 3851)
+++ remoting2/branches/2.x/src/main/org/jboss/remoting/transport/rmi/RemotingRMIClientSocketFactory.java	2008-04-02 04:25:01 UTC (rev 3852)
@@ -42,7 +42,6 @@
 import org.jboss.remoting.InvokerLocator;
 import org.jboss.remoting.Remoting;
 import org.jboss.remoting.Version;
-import org.jboss.remoting.loading.ClassByteClassLoader;
 
 
 /**
@@ -298,13 +297,26 @@
       private int port;
       private int hashCode;
 
-      public ComparableHolder(InvokerLocator invokerLocator)
+      public ComparableHolder(final InvokerLocator invokerLocator)
       {
          protocol = invokerLocator.getProtocol().toLowerCase();
          
          try
          {
-            host = InetAddress.getByName(invokerLocator.getHost());
+            try
+            {
+               host = (InetAddress)AccessController.doPrivileged( new PrivilegedExceptionAction()
+               {
+                  public Object run() throws UnknownHostException
+                  {
+                     return InetAddress.getByName(invokerLocator.getHost());
+                  }
+               });
+            }
+            catch (PrivilegedActionException e)
+            {
+               throw (UnknownHostException) e.getCause();
+            }  
          }
          catch (UnknownHostException e)
          {