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

Valerie Peng valeriep at openjdk.java.net
Thu Oct 8 20:49:29 UTC 2020


On Thu, 8 Oct 2020 06:51:08 GMT, Anthony Scarpino <ascarpino at openjdk.org> wrote:

>> 8253821: Improve ByteBuffer performance with GCM
>
> Anthony Scarpino has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Xuelei comments

src/java.base/share/classes/com/sun/crypto/provider/GaloisCounterMode.java line 432:

> 430:     // For input it takes the ibuffer which is wrapped in 'buffer' and 'src'
> 431:     // from doFinal.
> 432:     void doLastBlock(ByteBuffer buffer, ByteBuffer src, ByteBuffer dst)

The ordering of these new ByteBuffer-arg methods seems random? Perhaps, either group them altogether or move them so
that they are together with the byte[] counterpart methods?

src/java.base/share/classes/com/sun/crypto/provider/GaloisCounterMode.java line 508:

> 506:     }
> 507:
> 508:     int decrypt(ByteBuffer src, ByteBuffer dst) {

Similar to above. It seems a bit strange to see decrypt(...) method in between encrypt(...) methods.

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

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



More information about the security-dev mailing list