Author: remm
Date: Wed Sep  6 04:10:54 2006
New Revision: 440687

URL: http://svn.apache.org/viewvc?view=rev&rev=440687
Log:
- Oops, the order of evaluation is most likely wrong (accept needs to be done 
first, since it may throw an Error)
  compared to the TC 6 branch.

Modified:
    
tomcat/connectors/trunk/util/java/org/apache/tomcat/util/net/AprEndpoint.java

Modified: 
tomcat/connectors/trunk/util/java/org/apache/tomcat/util/net/AprEndpoint.java
URL: 
http://svn.apache.org/viewvc/tomcat/connectors/trunk/util/java/org/apache/tomcat/util/net/AprEndpoint.java?view=diff&rev=440687&r1=440686&r2=440687
==============================================================================
--- 
tomcat/connectors/trunk/util/java/org/apache/tomcat/util/net/AprEndpoint.java 
(original)
+++ 
tomcat/connectors/trunk/util/java/org/apache/tomcat/util/net/AprEndpoint.java 
Wed Sep  6 04:10:54 2006
@@ -997,10 +997,11 @@
                 }
 
                 try {
-                    // Allocate a new worker thread
                     // Accept the next incoming connection from the server 
socket
+                    long socket = Socket.accept(serverSock);
+                    // Allocate a new worker thread
                     // Hand this socket off to an appropriate processor
-                    getWorkerThread().assign(Socket.accept(serverSock), true);
+                    getWorkerThread().assign(socket, true);
                 } catch (Throwable t) {
                     log.error(sm.getString("endpoint.accept.fail"), t);
                 }



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to