[jdk17u-dev] RFR: 8314225: SIGSEGV in JavaThread::is_lock_owned [v2]

[Goetz Lindenmaier]

On Fri, 10 Jan 2025 10:53:49 GMT, Richard Reingruber <rrich at openjdk.org> wrote:

>> Goetz Lindenmaier has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix assertion message.
>
> src/hotspot/share/runtime/vframeArray.cpp line 98:
> 
>> 96:         } else {
>> 97:           assert(monitor->owner() != nullptr, "monitor owner must not be null");
>> 98:           assert(!monitor->owner()->is_unlocked() && !monitor->owner()->has_bias_pattern(), "object must be null or locked, and unbiased");
> 
> Suggestion:
> 
>           assert(!monitor->owner()->is_unlocked() && !monitor->owner()->has_bias_pattern(), "object must be locked, and unbiased");

Thanks for catching this, makes sense.  Fixed.

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

PR Review Comment: https://git.openjdk.org/jdk17u-dev/pull/3177#discussion_r1910217971