RFR: 8283022: com/sun/crypto/provider/Cipher/AEAD/GCMBufferTest.java failing with -Xcomp after 8273297

Alan Bateman alanb at openjdk.java.net
Mon Apr 18 09:01:36 UTC 2022


On Mon, 18 Apr 2022 05:06:26 GMT, Smita Kamath <svkamath at openjdk.org> wrote:

> When input length provided to the intrinsic is 8192, only 7680 bytes are processed as the intrinsic operates on multiples of 768 bytes.
> In implGCMCrypt(ByteBuffer src, ByteBuffer dst) method, 
> dst.put(bout, 0, PARALLEL_LEN) statement caused the ciphertext mismatch as PARALLEL_LEN was set to 8192. 
> Since the intrinsic only processed 7680 bytes, the rest output was incorrect.

It's good that this issue has been found. There seems to be an intrinsic for aarch64 with a vectorized GCM implementation, I guess is must also work in multiples of 768 bytes, so is this change okay there too?

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

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



More information about the security-dev mailing list