[8u] RFR: 8257997: sun/security/ssl/SSLSocketImpl/SSLSocketLeak.java again reports leaks after JDK-8257884
Alex Kashchenko
akashche at redhat.com
Tue Apr 20 19:52:17 UTC 2021
Hi,
Please review the backport of JDK-8257997 to 8u:
Bug: https://bugs.openjdk.java.net/browse/JDK-8257997
Original changeset: https://github.com/openjdk/jdk/commit/29ffffa7
8u webrev: https://cr.openjdk.java.net/~akasko/jdk8u/8257997/webrev.00/
8u exported commit:
https://cr.openjdk.java.net/~akasko/jdk8u/8257997/8257997_8u.changeset
This change is a follow-up to JDK-8256818 8u backport [1] and needs to
be applied on top of it.
Note 1, webrev was generated with "-r" flag, that doesn't include
commit metadata, so there is a separate exported commit link above.
Note 2, in mainline jdk there were two more changes between
JDK-8256818 and JDK-8257997:
JDK-8257670: sun/security/ssl/SSLSocketImpl/SSLSocketLeak.java reports leaks [2]
JDK-8257884: Re-enable
sun/security/ssl/SSLSocketImpl/SSLSocketLeak.java as automatic test
[3]
These changes are included into the proposed backport. JDK-8257670
(switch the test from "othervm" to "manual") won't apply to 8u,
because JDK-8256818 requires native lib and was already marked as
"manual". And JDK-8257884 changes the same lines as the proposed fix.
Testing: checked that changed test passes on Windows and Linux.
[1] https://mail.openjdk.java.net/pipermail/jdk8u-dev/2021-April/013660.html
[2] https://bugs.openjdk.java.net/browse/JDK-8257670
[3] https://bugs.openjdk.java.net/browse/JDK-8257884
--
-Alex
More information about the jdk8u-dev
mailing list