[8u] RFR: 8257670: sun/security/ssl/SSLSocketImpl/SSLSocketLeak.java reports leaks

Hohensee, Paul hohensee at amazon.com
Tue Apr 27 14:57:10 UTC 2021


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:10 AM
To: jdk8u-dev <jdk8u-dev at openjdk.java.net>
Subject: [8u] RFR: 8257670: sun/security/ssl/SSLSocketImpl/SSLSocketLeak.java reports leaks

Hi,

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

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

Original change: https://github.com/openjdk/jdk/commit/39b8a2e6

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

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

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


This change is a follow-up to JDK-8256818 (review thread [1]) and
needs to be applied on top of it. The change to the jtreg @run tag is
not applicable to 8u, because the test requires a native lib and is
already marked as manual in 8u.

Testing: checked that changed test passes.


[1] https://mail.openjdk.java.net/pipermail/jdk8u-dev/2021-April/013695.html

--
-Alex




More information about the jdk8u-dev mailing list