On Mon, 30 Jun 2025 13:03:23 GMT, Artem Semenov <aseme...@openjdk.org> wrote:
>> The defect has been detected and confirmed in the function >> ```IterateOverHeapObjectClosure::do_object()``` located in the file >> ```src/hotspot/share/prims/jvmtiTagMap.cpp``` with static code analysis. >> This defect can potentially lead to a null pointer dereference. >> >> The pointer ```oop o``` is passed to the constructor of the CallbackWrapper >> class, where it is dereferenced without a null check. > > Artem Semenov has updated the pull request incrementally with one additional > commit since the last revision: > > changed if tu assert The fix looks good, but I assume that the static analysis tool that reported the false warning is still going to report it, or at least do so in product builds where the assert code is not included. ------------- Marked as reviewed by cjplummer (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/26002#pullrequestreview-2973147893