RFR: 8317294: Classloading throws exceptions over already pending exceptions [v2]
Aleksey Shipilev
shade at openjdk.org
Tue Oct 3 21:20:19 UTC 2023
> See the bug for reproducer. I think we are just missing `return` after throwing the previous exception in `ClassFileParser::check_super_interface_access`. So we re-enter the method again with another exception in hand and fail the assert. This path is special, because we are looping, and we really need to exit after the first exception.
>
> It is not easy to add a regression test that pokes into super-interfaces from some JDK internals.
>
> Additional testing:
> - [x] CTW reproducer no longer fails
> - [x] Linux AArch64 `tier1 tier2 tier3` fastdebug
Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
Return near the throw
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/15984/files
- new: https://git.openjdk.org/jdk/pull/15984/files/2b78e5b0..842c2801
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=15984&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=15984&range=00-01
Stats: 3 lines in 1 file changed: 2 ins; 1 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/15984.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/15984/head:pull/15984
PR: https://git.openjdk.org/jdk/pull/15984
More information about the hotspot-runtime-dev
mailing list