On Wed, 21 Sep 2022 02:26:02 GMT, David Holmes <[email protected]> wrote:
>> Thomas Stuefe has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> cjplummer feedback
>
> src/hotspot/share/classfile/classLoaderHierarchyDCmd.cpp line 209:
>
>> 207: st->print(" \"%s\",", the_loader_name);
>> 208: }
>> 209: st->print(" %s", the_loader_class_name);
>
> `the_loader_class_name` could be NULL here. You need to change
> `loader_class_name()` t return "??" like the original code uses.
Good catch. We never encounter this because Klass can only be null for the
bootstrap loader, which is handled in a different branch. I'll fix it.
-------------
PR: https://git.openjdk.org/jdk/pull/10312
