[8u] RFR: 8257997: sun/security/ssl/SSLSocketImpl/SSLSocketLeak.java again reports leaks after JDK-8257884

[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:12 AM
To: jdk8u-dev <jdk8u-dev at openjdk.java.net>
Subject: Re: [8u] RFR: 8257997: sun/security/ssl/SSLSocketImpl/SSLSocketLeak.java again reports leaks after JDK-8257884

On 4/20/21, Alex Kashchenko <akashche at redhat.com> wrote:
> 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/
>
> [...]
>

Updated webrev following up the comments from JDK-8256818 review thread [1]:

8u webrev: https://cr.openjdk.java.net/~akasko/jdk8u/8257997/webrev.01/

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


This change is a follow-up to JDK-8256818 (review thread [1]),
JDK-8257670 [2] and  JDK-8257884 [3] and needs to be applied on top of
them.

Testing: checked that changed test passes on Windows and Linux.


[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-8257884

--
-Alex