RFR: 8253821: Improve ByteBuffer performance with GCM [v3]
Anthony Scarpino
ascarpino at openjdk.java.net
Tue Nov 3 20:42:00 UTC 2020
On Tue, 3 Nov 2020 01:07:12 GMT, Anthony Scarpino <ascarpino at openjdk.org> wrote:
>> src/java.base/share/classes/com/sun/crypto/provider/GCTR.java line 153:
>>
>>> 151: while (processed > MAX_LEN) {
>>> 152: encrypt(in, offset, MAX_LEN, out, 0);
>>> 153: dst.get(out, 0, MAX_LEN);
>>
>> Shouldn't this be "put" instead of "get"?
>
> Yeah.. I'm surprised that wasn't caught by the tests. I will look to see what case I need to make to check that.
So there is only one calling method that never gives this method it more than 16 bytes. That is why testing never caught this. But it doesn't hurt performance leaving the ability to call larger input sizes.
-------------
PR: https://git.openjdk.java.net/jdk/pull/411
More information about the security-dev
mailing list