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