RFR: 8367561: Getting some "header" property from a file:// URL causes a file descriptor leak [v3]

Daniel Fuchs dfuchs at openjdk.org
Mon Oct 13 11:18:07 UTC 2025


On Mon, 13 Oct 2025 11:15:16 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Hello Volkan, I've now added a comment in the test methods to explain why this line is present.
>
> Technically the reachabilityFence should be in the finally clause of a try-finally to make sure it's not rearranged away by JIT - but I'm not sure how important this is for a test like this. Alternative could be to stash the reference in a static field. As long as the test methods are not run in parallel that should work too.

Otherwise LGTM

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27633#discussion_r2426004480


More information about the net-dev mailing list