RFR: 8315739: Missing null check in os::vm_min_address [v2]

Jorn Vernee jvernee at openjdk.org
Thu Sep 21 12:40:43 UTC 2023


On Tue, 12 Sep 2023 05:26:25 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

>> Please review this fix for handling of file `/proc/sys/vm/mmap_min_addr` which does not exist on WSL1.
>> 
>> This fixes a crash if the file is absent, and a file handle leak (missing `fclose`) when the file is present.
>> 
>> Linux tier1 tests clean.
>
> Daniel Jeliński has updated the pull request incrementally with one additional commit since the last revision:
> 
>   move MAX after fclose

Looks good. (Works like a charm on my machine as well. Thanks!)

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

Marked as reviewed by jvernee (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/15596#pullrequestreview-1637716900


More information about the hotspot-runtime-dev mailing list