On Thu, 29 May 2025 19:53:50 GMT, Leonid Mesnik <lmes...@openjdk.org> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one >> additional commit since the last revision: >> >> review: add a call to ReleaseByteArrayElements to new function >> jni_array_to_jvmti_allocated > > test/lib/jdk/test/lib/jvmti/jvmti_common.hpp line 458: > >> 456: fatal(jni, "JNI GetArrayLength returned a non-positive value"); >> 457: } >> 458: jbyte* jni_arr = jni->GetByteArrayElements(arr, nullptr); > > I think you need to release array before exit from this function to unpin it. Good catch, thanks. Fixed now. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/25422#discussion_r2115161303
