On 23/03/2020 18:07, Doerr, Martin wrote:
Hi,
I'd like to backport JDK-8223326 from jdk/jdk.
11u backport issue:
https://bugs.openjdk.java.net/browse/JDK-8241460
Original change:
https://hg.openjdk.java.net/jdk/jdk/rev/29624901d8bc
11u backport webrev:
http://cr.openjdk.java.net/~mdoerr/8223326_nio_socket_11u/webrev.00/
I had to integrate the change manually due to context changes. However, the new
code fits to the 11u versions.
Please review.
Socket(SocketImpl) is only specified to do a permission check when the
impl is non-null. The socket adaptor in JDK 12 and older releases
doesn't have a dummy impl so the change should not be needed. If there
is a security exception thrownhere then it suggests something may be
broken elsewhere, do you have a stack trace?
-Alan