RE: [PATCH] SOCK_CLOEXEC for opening sockets

2018-07-30 Thread Andrew Luo
Fixed a bug, updated (I was calling accept directly instead of a helper function that calls accept then fcntl): diff --git a/make/lib/Lib-jdk.net.gmk b/make/lib/Lib-jdk.net.gmk --- a/make/lib/Lib-jdk.net.gmk +++ b/make/lib/Lib-jdk.net.gmk @@ -33,9 +33,11 @@ NAME := extnet, \ OPTIMIZ

RE: [PATCH] SOCK_CLOEXEC for opening sockets

2018-07-30 Thread Andrew Luo
I've updated my patch. Let me know if this approach looks good to you. I'm using dlsym to get the function pointer to accept4 at runtime so we can support compiling on newer Linux systems and using the binaries on older Linux systems (where the newer Linux system has accept4, and the older Lin

Re: Code Review Request, JDK-8207009 SSLEngine#closeInbound mentions SSLException when no close_notify is received

2018-07-30 Thread Xuelei Fan
Please let me know your concerns by the end of August 1st, 2018. Thanks, Xuelei On 7/30/2018 9:59 AM, Xuelei Fan wrote: Hi, Please review the update for the TLS 1.3 half-close and synchronization implementation:    http://cr.openjdk.java.net/~xuelei/8207009/webrev.00/ Unlike TLS 1.2 and