RFR: 8253821: Improve ByteBuffer performance with GCM [v3]
Anthony Scarpino
ascarpino at openjdk.java.net
Mon Nov 2 22:48:03 UTC 2020
On Mon, 2 Nov 2020 17:51:03 GMT, Valerie Peng <valeriep at openjdk.org> wrote:
>> Anthony Scarpino has updated the pull request incrementally with six additional commits since the last revision:
>>
>> - style
>> - style & comments
>> - full update
>> - remove old
>> - update
>> - outputsize
>
> src/java.base/share/classes/com/sun/crypto/provider/CipherCore.java line 726:
>
>> 724:
>> 725: // check output buffer capacity
>> 726: if (getMode() != GCM_MODE && (output == null ||
>
> Is the buffer size for GCM checked somewhere else? For decryption, it's probably ok to skip the check as no output is returned until doFinal(...). For encryption, there is still a chance for ShortBufferException? Javadoc states that for ShortBufferException, the call can be re-tried with a larger buffer.
I know the short buffers were being checked, but I removed the mode check, the tests still pass fine, so I figured this must be a unnecessary change.
-------------
PR: https://git.openjdk.java.net/jdk/pull/411
More information about the security-dev
mailing list