On Fri, 19 Nov 2021 10:15:05 GMT, Serguei Spitsyn <sspit...@openjdk.org> wrote:

>> src/hotspot/share/prims/jvmtiEnvBase.cpp line 1625:
>> 
>>> 1623:     return; /* JVMTI_ERROR_THREAD_NOT_ALIVE (default) */
>>> 1624:   }
>>> 1625:   assert(_state->get_thread() == java_thread, "Must be");
>> 
>> The `assert()` on L1625 is subject to the same race as the original site.
>> This `is_exiting()` check is made under the protection of the
>> `JvmtiThreadState_lock` so it is sufficient to protect that `assert()`.
>
> Okay, thanks!

Same comment as above.

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

PR: https://git.openjdk.java.net/jdk/pull/6440

Reply via email to