On Wed, 31 Aug 2022 21:47:05 GMT, Serguei Spitsyn <[email protected]> wrote:
> The problem is that the following assert in the JvmtiExport::post_object_free
> is wrong:
> ` assert(env->is_enabled(JVMTI_EVENT_OBJECT_FREE), "checking");`
>
> Even though the condition was checked before, it can be changed later as it
> is racy by JVM TI design.
> It has to be replaced with the check:
>
> if (!env->is_enabled(JVMTI_EVENT_OBJECT_FREE)) {
> return; // the event type has been already disabled
> }
>
>
> In progress: mach5 nsk.jvmti and nsk.jdi test runs.
This pull request has now been integrated.
Changeset: 99c3ab01
Author: Serguei Spitsyn <[email protected]>
URL:
https://git.openjdk.org/jdk/commit/99c3ab01773fcab885aa041345aab1a1ad4d852f
Stats: 3 lines in 1 file changed: 2 ins; 0 del; 1 mod
8293010: JDI ObjectReference/referringObjects/referringObjects001 fails:
assert(env->is_enabled(JVMTI_EVENT_OBJECT_FREE)) failed: checking
Reviewed-by: cjplummer, zgu, lmesnik
-------------
PR: https://git.openjdk.org/jdk/pull/10109
