RFR: 8313816: Accessing jmethodID might lead to spurious crashes [v6]

Jaroslav Bachorik jbachorik at openjdk.org
Thu Nov 23 15:22:09 UTC 2023


On Mon, 20 Nov 2023 22:08:49 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> Jaroslav Bachorik has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add exhaustive check for method holder availability (1)
>
> src/hotspot/share/classfile/classFileParser.cpp line 5579:
> 
>> 5577: 
>> 5578:   if (_methods != nullptr) {
>> 5579:     // Free methods - those methods are not fully wired and miss the method holder
> 
> How about saying: for methods whose InstanceKlass as method holder is not yet created?

And is now back, but I applied your suggestion, @coleenp

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16662#discussion_r1403513054


More information about the hotspot-dev mailing list