RFR: 8361614: Missing sub-int value validation in the Class-File API [v2]

Chen Liang liach at openjdk.org
Tue Jul 8 20:28:23 UTC 2025


> In the `class` file format, a lot of the values are `u1` or `u2`; the Class-File API consistently model them with `int`. However, the API does not, in general, validate that int values passed to the factory methods are not out of the bounds as defined in the class file format. This patch proposes to add such validation for factory methods accepting such more narrow data, prepared by examining all `int`-accepting methods in the Class-File API.
> 
> I expect this to have a small compatibility impact - besides the -1 for the minor version, there is no other places where most significant bits are ever meaningful, and I special cased it and consistently fail fast for all other OOB values, which always mean programmer errors.
> 
> A CSR will be created soon as well.

Chen Liang has updated the pull request incrementally with one additional commit since the last revision:

  Fix style

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/26201/files
  - new: https://git.openjdk.org/jdk/pull/26201/files/74fd977d..67aa74ae

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=26201&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=26201&range=00-01

  Stats: 3 lines in 1 file changed: 0 ins; 2 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/26201.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/26201/head:pull/26201

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


More information about the core-libs-dev mailing list