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

Chen Liang liach at openjdk.org
Fri Aug 29 14:39:02 UTC 2025


On Fri, 29 Aug 2025 13:56:01 GMT, Chen Liang <liach at openjdk.org> wrote:

>> 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:
> 
>   Another missed spot

Thanks for the reviews!

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

PR Comment: https://git.openjdk.org/jdk/pull/26201#issuecomment-3237248590


More information about the core-libs-dev mailing list