On Wed, 12 Feb 2025 21:08:50 GMT, Valerie Peng <[email protected]> wrote:
>> Mark Powers has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains three commits:
>>
>> - Merge master
>> - review comments from Valerie and Sean
>> - first iteration
>
> src/java.base/share/classes/javax/crypto/spec/IvParameterSpec.java line 73:
>
>> 71: * or {@code (iv.length - offset < len)}
>> 72: * @throws ArrayIndexOutOfBoundsException if {@code offset}
>> 73: * or {@code len} index bytes outside the {@code iv}.
>
> `RC2ParameterSpec(int, byte[], int)` ctor should be documented with AIOOBE as
> well. If you pass a negative offset into it, AIOOBE is thrown by the
> `System.arraycopy()` call.
fixed
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23188#discussion_r1963680579
