[lworld] RFR: 8262746: [lworld] jcmd VM.class_print_layout crash due to NULL cld name

[Yi Yang]

On Tue, 2 Mar 2021 14:06:14 GMT, Frederic Parain <fparain at openjdk.org> wrote:

>> Hi, can I have a review of this one-line fix?
>> 
>> jcmd VM.class_print_layout crashes when printing classes which loaded by bootstrap loader since these classes has NULL class_loder_data()->name().
>> 
>> But actually, I wonder to know why we don't print the layout by modifications of existing functions in FieldLayoutBuilder::build_layout directly? It seems FieldLayoutBuilder can gather more complete layout information.
>> 
>> Thanks,
>> Yang
>
> Looks good to me.
> 
> Thank you for fixing it.
> 
> Fred

Thanks Frederic for the reviews. And thanks Tobias for creating a related JBS issue!

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

PR: https://git.openjdk.java.net/valhalla/pull/354