RFR: 8361447: [REDO] Checked version of JNI Release<type>ArrayElements needs to filter out known wrapped arrays
David Holmes
dholmes at openjdk.org
Tue Jul 8 05:08:40 UTC 2025
On Tue, 8 Jul 2025 05:01:46 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
>> The original PR (https://github.com/openjdk/jdk/pull/26123) had incorrect usage of `SafeFetch32`. This PR replaces that with the use of `os::is_readable_range` as proposed by @tstuefe (thank you!). In addition I found that code was no longer getting exercised by the regression test so I added a gtest for that (during which it was discovered that the new implementation in `verify` requires any gtest that uses it to be `TEST_VM` rather than `TEST`).
>>
>> Testing: tiers 1-4
>>
>> Thanks
>
> This looks good to me. Thank you for considering my suggestions.
Thanks for the review @tstuefe !
-------------
PR Comment: https://git.openjdk.org/jdk/pull/26177#issuecomment-3047372852
More information about the hotspot-dev
mailing list