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