[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