[jdk11u-dev] RFR: 8273553: sun.security.ssl.SSLEngineImpl.closeInbound also has similar error of JDK-8253368 [v2]

[Goetz Lindenmaier]

On Tue, 11 Oct 2022 14:35:58 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

>> I backport this for parity with 11.0.18-oracle.
>> 
>> 
>> I had to resolve closeInbound() in SSLEngineImpl.java because 
>> "8221882: Use fiber-friendly java.util.concurrent.locks in JSSE" 
>> is not in 11.
>
> Goetz Lindenmaier has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add try ... finally.

Hi, 
I am not sure whether closeInbound() is needed if isInboundDone() is true.
I placed it according to https://github.com/openjdk/jdk17u-dev/blob/master/src/java.base/share/classes/sun/security/ssl/SSLSocketImpl.java#L826
I don't think 8273553 will be backported, as it is a preparation for fibres / virtual threads.

-------------

PR: https://git.openjdk.org/jdk11u-dev/pull/1422