On Wed, 9 Aug 2023 23:09:37 GMT, Anthony Scarpino <[email protected]> wrote:
>> Liam Miller-Cushon has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Check for AEADBadTagException specifically
>
> test/jdk/com/sun/crypto/provider/Cipher/AEAD/GCMShortInput.java line 54:
>
>> 52: cipher.doFinal(ByteBuffer.allocate(0),
>> ByteBuffer.allocate(0));
>> 53: throw new AssertionError("GeneralSecurityException
>> expected");
>> 54: } catch (GeneralSecurityException e) {
>
> The test should be verifying the exception is AEADBadTagException, not
> GeneralSecurityException, since that is the issue.
Done
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15212#discussion_r1289345612
