On Tue, 29 Sep 2026 06:53:48 GMT, Sergey Bylokhov <[email protected]> wrote:

>> Add some missing 'up front' checks for GlyphVector access indices that are 
>> out of bounds.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> src/java.desktop/share/classes/sun/font/StandardGlyphVector.java line 433:
> 
>> 431:     // relative to gv origin
>> 432:     public Shape getGlyphOutline(int ix) {
>> 433:         if (ix < 0 || ix > glyphs.length) {
> 
> per the spec in the parent class it should be ">="?
>>     * @throws IndexOutOfBoundsException if {@code glyphIndex}
>>     *   is less than 0 or greater than or equal to the number
>>     *   of glyphs in this {@code GlyphVector}

btw seems the test does not cover some corner cases?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/33078#discussion_r4130492813

Reply via email to