[RFR] 8166597: Crypto support for the EdDSA Signature Algorithm (JEP 339)
Anthony Scarpino
anthony.scarpino at oracle.com
Fri Mar 6 19:27:28 UTC 2020
On 3/6/20 10:22 AM, Sean Mullan wrote:
> It will take me a few iterations to review all of this, but here are
> some comments so far, mostly minor things:
>
> - src/java.base/share/classes/java/security/spec/NamedParameterSpec.java
>
> Need "@since 15" on the two new constants. (Assuming this goes in 15).
Ok
>
> - src/java.base/share/classes/sun/security/util/ArrayUtil.java
>
> 35 * This class holds various utility methods for arrays.
>
> The previous comment sounded more specific, why did you change it?
>
I don't know why Adam changed that. I'll undo it, seem pointless
> - src/java.base/share/classes/sun/security/x509/AlgorithmId.java
>
> 906 public static ObjectIdentifier ed25519_oid =
> 907 ObjectIdentifier.of("1.3.101.112");
> 908 public static ObjectIdentifier ed448_oid =
> 909 ObjectIdentifier.of("1.3.101.113");
>
> Make them final.
ok
>
> --Sean
>
> On 2/25/20 3:49 PM, Anthony Scarpino wrote:
>> Hi
>>
>> I need a code review for the EdDSA support in JEP 339. The code
>> builds on the existing java implemented constant time classes used for
>> XDH and the NIST curves. The change also adds classes to the public
>> API to support EdDSA operations.
>>
>> All information about the JEP is located at:
>> JEP 339: https://bugs.openjdk.java.net/browse/JDK-8199231
>> CSR: https://bugs.openjdk.java.net/browse/JDK-8190219
>>
>> webrev: https://cr.openjdk.java.net/~ascarpino/8166597/webrev/
>>
>> thanks
>>
>> Tony
More information about the security-dev
mailing list