On Thu, 7 Nov 2024 00:47:33 GMT, Ben Perez <bpe...@openjdk.org> wrote:
>> Java implementation of ML-KEM, the [FIPS >> 203](https://csrc.nist.gov/pubs/fips/203/final) post-quantum KEM scheme. >> Depends on https://github.com/openjdk/jdk/pull/21167 > > Ben Perez has updated the pull request incrementally with two additional > commits since the last revision: > > - default random for encaps, supported alg in SunJCE > - copyright header src/java.base/share/classes/com/sun/crypto/provider/ML_KEM.java line 500: > 498: byte[] kem_d, byte[] kem_z) > 499: throws NoSuchAlgorithmException, DigestException { > 500: var mlKemH = MessageDigest.getInstance("SHA3-256"); Should this algorithm be pulled out into a constant? Keccak is new-ish, but it may not be this forever and/or the output size may change from 256. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21478#discussion_r1833020568