[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