On Tue, 9 May 2023 12:45:27 GMT, Ferenc Rakoczi <[email protected]> wrote:
>> Implement support for Leighton-Micali Signatures (LMS) as described in RFC
>> 8554. LMS is an approved software signing algorithm for CNSA 2.0, with
>> SHA-256/192 parameters recommended.
>
> Ferenc Rakoczi has updated the pull request incrementally with one additional
> commit since the last revision:
>
> agreeing with the newest review comments
src/java.base/share/classes/sun/security/provider/HSS.java line 101:
> 99: }
> 100:
> 101: static class LMSPublicKey implements Serializable {
Add a `writeReplace` method. Look at other `PublicKey` implementations for an
example.
src/java.base/share/classes/sun/security/provider/HSS.java line 540:
> 538: throw new ProviderException("Digest implementation not
> found", e);
> 539: }
> 540: byte[] result = new byte[md.getDigestLength()];
No need to allocate an array here.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13691#discussion_r1188643495
PR Review Comment: https://git.openjdk.org/jdk/pull/13691#discussion_r1188630397