-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added a P256 variant #97
Changes from 1 commit
bec5538
304a347
54e95c7
b18ab70
6d0a5b8
d01af43
64aed9d
0206e33
4e6541a
3d86dbe
c7cce41
7aa667f
e659308
cf867fb
32ac139
c2edf17
120e1f2
46fd923
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -253,6 +253,21 @@ sa-MLDSA65-Ed25519 SIGNATURE-ALGORITHM ::= | |
pk-MLDSA65-Ed25519 } | ||
|
||
|
||
-- TODO: OID to be replaced by IANA | ||
id-MLDSA87-ECDSA-P256 OBJECT IDENTIFIER ::= { | ||
joint-iso-itu-t(2) country(16) us(840) organization(1) | ||
entrust(114027) algorithm(80) composite(8) signature(1) 71 } | ||
|
||
pk-MLDSA87-ECDSA-P256 PUBLIC-KEY ::= | ||
pk-CompositeSignature{ id-MLDSA87-ECDSA-P384, | ||
CompositeMLDSAPublicKey} | ||
|
||
sa-MLDSA87-ECDSA-P256 SIGNATURE-ALGORITHM ::= | ||
sa-CompositeSignature{ | ||
id-MLDSA87-ECDSA-P256, | ||
pk-MLDSA87-ECDSA-P256 } | ||
|
||
|
||
-- TODO: OID to be replaced by IANA | ||
id-MLDSA87-ECDSA-P384 OBJECT IDENTIFIER ::= { | ||
joint-iso-itu-t(2) country(16) us(840) organization(1) | ||
|
@@ -417,6 +432,24 @@ sa-HashMLDSA65-RSA4096-PKCS15-SHA512 SIGNATURE-ALGORITHM ::= | |
id-HashMLDSA65-RSA4096-PKCS15-SHA512, | ||
pk-HashMLDSA65-RSA4096-PKCS15-SHA512 } | ||
|
||
|
||
-- TODO: OID to be replaced by IANA | ||
id-HashMLDSA65-ECDSA-P256-SHA512 OBJECT IDENTIFIER ::= { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is it intentional to use SHA-512 with ECDSA P-256 (which usually uses P-256)? The SHA-512 hash will be truncated to 32 bytes (This is not the same as SHA-512/256). Regarding speed of SHA-256 versus SHA-512, on 32-bit SHA-256 is faster, and on 64-bit SHA-512 is faster, unless there is hardware SHA-256, in which case SHA-256 is faster. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is intentional. The full specification of this cipher suite is:
So the EC part is Complicated, but that's how we got here. This is intentional. |
||
joint-iso-itu-t(2) country(16) us(840) organization(1) | ||
entrust(114027) algorithm(80) composite(8) signature(1) 82 } | ||
|
||
pk-HashMLDSA65-ECDSA-P256-SHA512 PUBLIC-KEY ::= | ||
pk-CompositeSignature{ id-HashMLDSA65-ECDSA-P256-SHA512, | ||
CompositeMLDSAPublicKey} | ||
|
||
sa-HashMLDSA65-ECDSA-P256-SHA512 SIGNATURE-ALGORITHM ::= | ||
sa-CompositeSignature{ | ||
id-HashMLDSA65-ECDSA-P256-SHA512, | ||
pk-HashMLDSA65-ECDSA-P256-SHA512 } | ||
|
||
|
||
|
||
|
||
-- TODO: OID to be replaced by IANA | ||
id-HashMLDSA65-ECDSA-P384-SHA512 OBJECT IDENTIFIER ::= { | ||
joint-iso-itu-t(2) country(16) us(840) organization(1) | ||
|
@@ -426,7 +459,7 @@ pk-HashMLDSA65-ECDSA-P384-SHA512 PUBLIC-KEY ::= | |
pk-CompositeSignature{ id-HashMLDSA65-ECDSA-P384-SHA512, | ||
CompositeMLDSAPublicKey} | ||
|
||
sa-HashMLDSA65-ECDSA-P256-SHA512 SIGNATURE-ALGORITHM ::= | ||
sa-HashMLDSA65-ECDSA-P384-SHA512 SIGNATURE-ALGORITHM ::= | ||
sa-CompositeSignature{ | ||
id-HashMLDSA65-ECDSA-P384-SHA512, | ||
pk-HashMLDSA65-ECDSA-P384-SHA512 } | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be ML-DSA-65 instead of ML-DSA-87? The other changes seem to be for ML-DSA-65+ECDSA-P256 combo (e.g., id-HashMLDSA65-ECDSA-P256-SHA512).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops. I must have been tired or drunk when I wrote this. Good catch.