RFR: 8331008: KDF Implementation [v3]
Kevin Driver
kdriver at openjdk.org
Thu May 9 21:01:11 UTC 2024
On Thu, 9 May 2024 20:31:30 GMT, Weijun Wang <weijun at openjdk.org> wrote:
>> Kevin Driver has updated the pull request incrementally with one additional commit since the last revision:
>>
>> some code review comments
>
> src/java.base/share/classes/com/sun/crypto/provider/HkdfKeyDerivation.java line 223:
>
>> 221: // A switch would be nicer, but we may need to backport this before JDK 17
>> 222: // Also, JEP 305 came out in JDK 14, so we can't declare a variable in instanceof either
>> 223: if (kdfParameterSpec instanceof HKDFParameterSpec.Extract) {
>
> `kdfParameterSpec instanceof HKDFParameterSpec.Extract anExtract`.
Please read the comment. Avoiding declaring a variable during an `instanceof` check was intentional, since this will aid with backporting. I'm not sure how far back we'll have to port this.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18924#discussion_r1595981033
More information about the security-dev
mailing list