RFR: 8314225: SIGSEGV in JavaThread::is_lock_owned [v3]

[David Holmes]

On Wed, 1 May 2024 21:40:00 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

>> Not sure how `obj` can be null in this code. ???
>
> That is fetching from an index in the MonitorChunk* _monitors, so if we recognise null in element->fill_in() when populating MonitorChunk*, seems good to recognise it here in vframeArrayElement::unpack_on_stack()?

I can follow that logic but ... if it is null then what is this code actually doing? We have determined that the frame does contain locked monitors and so we are transferring them across. How can such a locked monitor have a null object?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18940#discussion_r1586877095