RFR [11] 8198358 : Align organization of DualStackPlainSocketImpl with TwoStacksPlainSocketImp [win]

Chris Hegarty chris.hegarty at oracle.com
Sun Mar 25 13:37:12 UTC 2018


> On 23 Mar 2018, at 21:55, Chris Hegarty <chris.hegarty at oracle.com> wrote:
> 
>> For reference, here's the combined webrev:
>> http://cr.openjdk.java.net/~igerasim/8198358/03/webrev/ <http://cr.openjdk.java.net/~igerasim/8198358/03/webrev/>
> 
> I think this is good. And thanks for the additional test coverage, this is great.

Actually, there is a couple of places in the new native code that
has the following pattern:

     if (sa.sa.sa_family != AF_INET) {
         JNU_ThrowByName(env, JNU_JAVANETPKG "SocketException",
                         "Protocol family not supported");
         NET_SocketClose(newfd);
         return -1;
   }

I would like to remove the NET_SocketClose, and allow the
exception propagating at the Java-level be responsible for 
closing the socket, which will then do so in cooperation
with the socket cleaner ( it will effectively cancel the cleaner ).

-Chris.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/net-dev/attachments/20180325/c9caa1a1/attachment.html>


More information about the net-dev mailing list