RFR: 8302225: SunJCE Provider doesn't validate key sizes when using 'constrained' transforms for AES/KW and AES/KWP
Xue-Lei Andrew Fan
xuelei at openjdk.org
Wed Feb 15 05:33:42 UTC 2023
On Wed, 15 Feb 2023 02:23:31 GMT, Valerie Peng <valeriep at openjdk.org> wrote:
> Due to an error in the existing regression test, this bug remain undiscovered until now. Added the key size check to the KeyWrapCipher class and fixed the regression test.
>
> Please help review this trivial fix.
>
> Thanks in advance,
> Valerie
src/java.base/share/classes/com/sun/crypto/provider/KeyWrapCipher.java line 302:
> 300: if (fixedKeySize != -1 && fixedKeySize != keyBytes.length) {
> 301: throw new InvalidKeyException("Invalid key length: " +
> 302: keyBytes.length + " bytes, should be " + fixedKeySize);
Is it necessary to check that the key algorithm is "AES", and the size is a valid AES key size, as the code in engineGetKeySize()?
-------------
PR: https://git.openjdk.org/jdk/pull/12569
More information about the security-dev
mailing list