On Mon, 27 Apr 2026 20:27:57 GMT, Sergey Bylokhov <[email protected]> wrote:
>> Phil Race has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8377568
>
> src/java.desktop/share/classes/java/awt/image/DataBuffer.java line 623:
>
>> 621: throw new ArrayIndexOutOfBoundsException("Index cannot be
>> negative : " + i);
>> 622: }
>> 623: if ((offsets[bank] + i) < i) {
>
> Should the bank be validated first? or it is assumed always correct in this
> place?
Yeah, `checkIndex(int bank, int i)` should call `checkBank` first.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29766#discussion_r3155708080
