RFR: 8366498: Wrong InstanceKlass::cast() in ClassFileParser::parse_super_class
David Holmes
dholmes at openjdk.org
Mon Sep 1 06:29:41 UTC 2025
On Mon, 1 Sep 2025 05:19:43 GMT, Ioi Lam <iklam at openjdk.org> wrote:
> Please review this small change.
>
> It affects debug build only, and I am not even sure if `cp->resolved_klass_at()` is ever called. Since we are very early in class file parsing, it's unlikely that any klass entries have been resolved at this point.
>
> Anyway, for correctness, the `InstanceKlass::cast()` should be done after the check for array class.
This does correct the cast but as you suggest this code should actually be unreachable - `is_klass()` will fail as no classes have been resolved at this stage of classfile parsing,
-------------
PR Review: https://git.openjdk.org/jdk/pull/27026#pullrequestreview-3172246976
More information about the hotspot-runtime-dev
mailing list