On 25/07/12 19:37, Xuelei Fan wrote:
Looks fine to me.
Looks fine to me too.
-Chris.
Thanks,
Xuelei
On 7/26/2012 9:49 AM, Seán Coffey wrote:
Requesting a review for the following issue :
http://bugs.sun.com/view_bug.do?bug_id=7179879
The SSLSocketImpl doesn't appear to handle the case where a connect
operation is cancelled
before an SSL session has begun (we simple ignore such a case)
Fix would be to honour the close call which interrupts the connect
request and close out the socket with the appropriate socket closed
exception.
webrev : http://cr.openjdk.java.net/~coffeys/webrev.7179879.jdk8/
<http://cr.openjdk.java.net/%7Ecoffeys/webrev.7179879.jdk8/>
I've run the testcase manually and verified the fix. I've attempted to
automate the testcase but had issue in getting an SSLServerSocket to not
over eagerly accept a client socket connect request (client connect
succeeding even before server code makes an accept call)
Regards,
Sean.