RFR: 8282306: os::is_first_C_frame(frame*) crashes on invalid link access [v2]

[Johannes Bechberger]

On Wed, 23 Feb 2022 21:58:59 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Johannes Bechberger has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Improve use of C macros
>
> src/hotspot/share/runtime/os.cpp line 1227:
> 
>> 1225:          !t->is_in_full_stack((address)fr->fp()) ||
>> 1226:          !t->is_in_full_stack((address)fr->sender_sp()) ||
>> 1227:          !t->is_in_full_stack((address)fr->link());
> 
> Isn't this check of `fr.link()` what you already did in `can_access_link`?

You're correct.

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

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