RFR: 8253821: Improve ByteBuffer performance with GCM [v5]

Anthony Scarpino ascarpino at openjdk.java.net
Wed Dec 2 02:39:10 UTC 2020


On Tue, 1 Dec 2020 22:42:54 GMT, Valerie Peng <valeriep at openjdk.org> wrote:

>> Anthony Scarpino has updated the pull request incrementally with five additional commits since the last revision:
>> 
>>  - test updates
>>  - test check mixup
>>  - Overlap protection
>>  - Updated code comments, all tests pass
>>  - Updated code comments, all tests pass
>
> src/java.base/share/classes/com/sun/crypto/provider/GaloisCounterMode.java line 539:
> 
>> 537:             // Construct and encrypt a block if there is enough 'buffer' and
>> 538:             // 'in' to make one
>> 539:             if ((inLen + remainder) >= blockSize) {
> 
> This big block of code is only needed when (remainder != 0), right?

If there is nothing in the ibuffer, then it never gets here.  Yes.

btw, make sure you're looking at the right webrev, the line numbers didn't match up with revision 4.

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

PR: https://git.openjdk.java.net/jdk/pull/411



More information about the security-dev mailing list