RFR: 8328874: Class::forName0 should validate the class name length early [v12]

[Guanqiang Han]

On Wed, 3 Sep 2025 18:40:26 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Guanqiang Han has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update Class.java
>>   
>>   change overflow check
>
>> /reviewers 2 reviewer
>> 
>> I recommend putting this PR on hold - 65535 encoded size for modified utf8 is significant enough for a cross-JDK utility. As @rose00 suggested, I think I will explore adding a check in ClassFile API, and sharing the check with here.
> 
> A common boolean function is useful and can be better optimized.
> The function should be in a common utility area, but the ClassFile API is a higher level API specific to class files.
> Most developers won't think to look there. The jdk.internal.util.ModifiedUtf isn't a bad place for both the boolean function and the utfLen function.

@RogerRiggs @liach  I’ve addressed the feedback in this PR. Please review again.

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

PR Comment: https://git.openjdk.org/jdk/pull/26802#issuecomment-3264157849