[11u] RFR(S): 8223326: Regression introduced by CPU sync: java.security.AccessControlException: access denied ("java.net.NetPermission" "setSocketImpl")

[Alan Bateman]

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