[jboss-remoting-commits] JBoss Remoting SVN: r3877 - remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/socketpool.

jboss-remoting-commits at lists.jboss.org jboss-remoting-commits at lists.jboss.org
Wed Apr 2 01:41:45 EDT 2008


Author: ron.sigal at jboss.com
Date: 2008-04-02 01:41:45 -0400 (Wed, 02 Apr 2008)
New Revision: 3877

Modified:
   remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/socketpool/SocketClientPoolWithSemaphoreTestCase.java
Log:
JBREM-930: Copy List to avoid ConcurrentModificationException.

Modified: remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/socketpool/SocketClientPoolWithSemaphoreTestCase.java
===================================================================
--- remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/socketpool/SocketClientPoolWithSemaphoreTestCase.java	2008-04-02 05:39:41 UTC (rev 3876)
+++ remoting2/branches/2.x/src/tests/org/jboss/test/remoting/transport/socket/socketpool/SocketClientPoolWithSemaphoreTestCase.java	2008-04-02 05:41:45 UTC (rev 3877)
@@ -25,6 +25,7 @@
 import java.net.InetAddress;
 import java.net.ServerSocket;
 import java.net.Socket;
+import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.Iterator;
 import java.util.List;
@@ -442,7 +443,7 @@
       
       field = SocketServerInvoker.class.getDeclaredField("threadpool");
       field.setAccessible(true);
-      List threadpool = (List) field.get(serverInvoker);
+      List threadpool = new ArrayList((List) field.get(serverInvoker));
       it = threadpool.iterator();
       field = ServerThread.class.getDeclaredField("socket");
       field.setAccessible(true);




More information about the jboss-remoting-commits mailing list