I've been debugging various problems with Azureus.  This is the first of
several patches. 

This first patch stops us from closing a socket when we get an exception
during connect().  It is standard behaviour to get timeout exception
here for non-blocking socket connect().  We will never get a proper
connection if we close() the socket at this point.

Ok to apply?

AG



2005-12-26  Anthony Green  <[EMAIL PROTECTED]>

        * java/net/Socket.java (connect): Don't close the socket on
        exceptions.


--- java/net/Socket.java~       2005-11-16 04:03:51.000000000 -0500
+++ java/net/Socket.java        2005-12-26 10:07:43.000000000 -0500
@@ -437,25 +437,7 @@
     if (! isBound())
       bind(null);
 
-    try
-      {
-       getImpl().connect(endpoint, timeout);
-      }
-    catch (IOException exception)
-      {
-       close();
-       throw exception;
-      }
-    catch (RuntimeException exception)
-      {
-       close();
-       throw exception;
-      }
-    catch (Error error)
-      {
-       close();
-       throw error;
-      }
+    getImpl().connect(endpoint, timeout);
   }
 
   /**





_______________________________________________
Classpath-patches mailing list
Classpath-patches@gnu.org
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to