RFR: 8229867: Re-examine synchronization usages in http and https protocol handlers [v2]
Daniel Fuchs
dfuchs at openjdk.java.net
Mon Oct 12 13:50:31 UTC 2020
On Fri, 9 Oct 2020 13:22:08 GMT, Alan Bateman <alanb at openjdk.org> wrote:
>> Daniel Fuchs has updated the pull request incrementally with one additional commit since the last revision:
>>
>> 8229867: Re-examine synchronization usages in http and https protocol handlers
>>
>> Incorporated review feedback
>
> src/java.base/share/classes/sun/net/www/http/HttpCapture.java line 59:
>
>> 57: // Although accessing files could result in blocking operations,
>> 58: // HttpCapture is a corner case; there seem no urgent need to convert
>> 59: // this class to using java.util.concurrent.locks at this time.
>
> The updated patch looks good but I think this comment needs another iteration to ensure that it doesn't confuse future
> maintainers. You could drop it or else replace it with something simple that says that HttpCapture does blocking I/O
> operations while holding monitors but it's not a concern because it rarely used.
Updated as requested.
-------------
PR: https://git.openjdk.java.net/jdk/pull/558
More information about the security-dev
mailing list