Re: RFR: 8332486: ClassFile API ArrayIndexOutOfBoundsException with label metadata

2024-05-20 Thread Paul Sandoz
On Mon, 20 May 2024 08:37:49 GMT, Adam Sotona  wrote:

> Parsing of a specifically corrupted class file cause unexpected 
> `ArrayIndexOutOfBoundsException` during label inflation.
> This patch checks the valid range and throws expected 
> `IllegalArgumentException` instead.
> Relevant test is added.
> 
> Please review.
> 
> Thanks,
> Adam

Marked as reviewed by psandoz (Reviewer).

-

PR Review: https://git.openjdk.org/jdk/pull/19304#pullrequestreview-2066846853


RFR: 8332486: ClassFile API ArrayIndexOutOfBoundsException with label metadata

2024-05-20 Thread Adam Sotona
Parsing of a specifically corrupted class file cause unexpected 
`ArrayIndexOutOfBoundsException` during label inflation.
This patch checks the valid range and throws expected 
`IllegalArgumentException` instead.
Relevant test is added.

Please review.

Thanks,
Adam

-

Commit messages:
 - 8332486: ClassFile API ArrayIndexOutOfBoundsException with label metadata

Changes: https://git.openjdk.org/jdk/pull/19304/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk=19304=00
  Issue: https://bugs.openjdk.org/browse/JDK-8332486
  Stats: 19 lines in 2 files changed: 18 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/19304.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/19304/head:pull/19304

PR: https://git.openjdk.org/jdk/pull/19304