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