On Wed, 7 Jul 2021 10:15:19 GMT, Alexey Bakhtin <abakh...@openjdk.org> wrote:

>> Please review the fix for JDK-8268965.
>> 
>> The new jtreg test is added for the described issue.
>> sun/security/ssl and javax/net/ssl tests are passed
>
> Alexey Bakhtin has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Add read lock for inputRecord.deplete

Hi Xuelei,
Thank you for review. You are right. It is better to add readLock to exclude 
concurrent reading. Unfortunately I can not use appInput.deplete() here because 
of input stream already marked as closed (called from 
TransportContext.fatal(TransportContext.java:376)). So, I've added 
appInput.readLock around deplete()

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

PR: https://git.openjdk.java.net/jdk/pull/4520

Reply via email to