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

[Yi Yang]

On Mon, 1 Mar 2021 11:42:07 GMT, Yi Yang <github.com+5010047+kelthuzadx 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

This pull request has now been integrated.

Changeset: 6131196e
Author:    Yi Yang <qingfeng.yy at alibaba-inc.com>
Committer: Frederic Parain <fparain at openjdk.org>
URL:       https://git.openjdk.java.net/valhalla/commit/6131196e
Stats:     74 lines in 2 files changed: 73 ins; 0 del; 1 mod

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

Reviewed-by: fparain

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

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