[8u] RFR: 8257884: Re-enable sun/security/ssl/SSLSocketImpl/SSLSocketLeak.java as automatic test

[Hohensee, Paul]

Lgtm.

Thanks,
Paul

-----Original Message-----
From: jdk8u-dev <jdk8u-dev-retn at openjdk.java.net> on behalf of Alex Kashchenko <akashche at redhat.com>
Date: Saturday, April 24, 2021 at 6:11 AM
To: jdk8u-dev <jdk8u-dev at openjdk.java.net>
Subject: [8u] RFR: 8257884: Re-enable sun/security/ssl/SSLSocketImpl/SSLSocketLeak.java as automatic test

Hi,

Please review the backport of JDK-8257884 to 8u:

Bug: https://bugs.openjdk.java.net/browse/JDK-8257884

Original change: https://github.com/openjdk/jdk/commit/fae7961a

11u backport: https://hg.openjdk.java.net/jdk-updates/jdk11u-dev/rev/59a6aaef42a7

8u webrev: https://cr.openjdk.java.net/~akasko/jdk8u/8257884/webrev.00/

8u exported commit:
https://cr.openjdk.java.net/~akasko/jdk8u/8257884/8257884_8u.changeset


This change is a follow-up to JDK-8256818 (review thread [1]) and
JDK-8257670 [2] and needs to be applied on top of them. It requires
minor adjustemts, because jdk.test.lib.Platform API used there is not
available in 8u.

Testing: checked that changed test can be compiled, it fails in my
environment reporting found leaks, this problem with it is mitigated
with a follow-up change JDK-8257997 [3].

[1] https://mail.openjdk.java.net/pipermail/jdk8u-dev/2021-April/013695.html
[2] https://bugs.openjdk.java.net/browse/JDK-8257670
[3] https://bugs.openjdk.java.net/browse/JDK-8257997

--
-Alex