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

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

Author: ron.sigal at jboss.com
Date: 2008-04-04 18:36:36 -0400 (Fri, 04 Apr 2008)
New Revision: 3896

Modified:
   remoting2/branches/2.x/src/main/org/jboss/remoting/transport/servlet/web/ServerInvokerServlet.java
Log:
JBREM-934: Put call to MBeanServerFactory.findMBeanServer() in AccessController.doPrivileged() call.

Modified: remoting2/branches/2.x/src/main/org/jboss/remoting/transport/servlet/web/ServerInvokerServlet.java
===================================================================
--- remoting2/branches/2.x/src/main/org/jboss/remoting/transport/servlet/web/ServerInvokerServlet.java	2008-04-04 22:35:09 UTC (rev 3895)
+++ remoting2/branches/2.x/src/main/org/jboss/remoting/transport/servlet/web/ServerInvokerServlet.java	2008-04-04 22:36:36 UTC (rev 3896)
@@ -44,6 +44,7 @@
 import java.io.IOException;
 import java.lang.reflect.Method;
 import java.security.AccessController;
+import java.security.PrivilegedAction;
 import java.security.PrivilegedActionException;
 import java.security.PrivilegedExceptionAction;
 import java.util.Iterator;
@@ -301,8 +302,16 @@
          }
       }
       
-      for (Iterator i = MBeanServerFactory.findMBeanServer(null).iterator(); i.hasNext();)
+      Iterator i = (Iterator) AccessController.doPrivileged( new PrivilegedAction()
       {
+         public Object run()
+         {
+            return MBeanServerFactory.findMBeanServer(null).iterator();
+         }
+      });
+      
+      while(i.hasNext())
+      {
          MBeanServer server = (MBeanServer) i.next();
          
          if (server.getDefaultDomain() == null)