-
Notifications
You must be signed in to change notification settings - Fork 752
Add API support for External-MU Signature[ML-DSA] #2366
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
Merged
Merged
Changes from all commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
6d1aeea
add API for external-MU signing (ML_DSA)
abhi-dev-engg 0a76d38
code formatting
abhi-dev-engg 0cb70a3
fix issues
abhi-dev-engg 1f2dce4
add upstream changes to support extmu variant
abhi-dev-engg 63fd0eb
fix verify functions for x86
abhi-dev-engg 16a4a1b
avoid duplication of src compilation for mu variants
abhi-dev-engg 1852afd
fix multiple definition error
abhi-dev-engg 1048b1d
add missed upstream changes
abhi-dev-engg 83dcd4c
fix CMake indentation issues
abhi-dev-engg 3da8d2f
add check for msglen for mu variants
abhi-dev-engg 7881c95
upstream sig fixes
abhi-dev-engg 7e8b7e9
skip mem tests for extmu variants
abhi-dev-engg ec685fe
fix msg length issues for mu variant
abhi-dev-engg 55a647c
remove un-needed changes
abhi-dev-engg 7a5f741
Merge branch 'main' into extmu
abhi-dev-engg b23a039
review fixes & change mu tests from internal APIs to external
abhi-dev-engg 1ad115c
code formatting
abhi-dev-engg 628cc0c
fix build issues
abhi-dev-engg ec5d00a
remove unused param
abhi-dev-engg b45df7a
code formatting fix
abhi-dev-engg 4aa5c22
remove hardcoding of 64 as mu length
abhi-dev-engg e414a02
address more review comments
abhi-dev-engg 0d49427
remove kat changes for ext-mu variants
abhi-dev-engg 61223d9
Merge branch 'main' into extmu
abhi-dev-engg 257831c
dummy commit to restart tests
abhi-dev-engg 9a58b4b
dummy commit to trigger build
abhi-dev-engg b551830
fix more review comments
abhi-dev-engg d2ddedc
Merge branch 'main' into extmu
abhi-dev-engg 351a9c9
upstream fixes
abhi-dev-engg a9121db
fix code formatting
abhi-dev-engg 244a8e5
tabs instead of spaces
abhi-dev-engg c191438
upstream changes
abhi-dev-engg 577a812
code formatting
abhi-dev-engg d517d70
fix more code formatting
abhi-dev-engg 007b3db
Merge branch 'main' into extmu
abhi-dev-engg eda34f5
Merge branch 'main' into extmu
abhi-dev-engg ed4a65d
adapt latest wycheproof
abhi-dev-engg b562608
code formatting
abhi-dev-engg 2cc25db
check mslgen before hex conversion
abhi-dev-engg 1438786
code formatting
abhi-dev-engg 5369c59
step 1 : update latest yaml
abhi-dev-engg 6f5e8d7
merge main branch & resolve conflicts
abhi-dev-engg a858f14
add checks for extmu apis
abhi-dev-engg 5fa72e2
correct macro for sig-dsa
abhi-dev-engg f933a3e
Merge branch 'main' into extmu
abhi-dev-engg 1ac8d5c
add new line at end of cmake
abhi-dev-engg 07e1f02
minor cosmetic fix
abhi-dev-engg 1895f08
revert changes
abhi-dev-engg b2acd7a
Merge branch 'main' into extmu
abhi-dev-engg c640e64
Merge branch 'main' into extmu
xuganyu96 8e67a08
Merge branch 'main' into extmu
abhi-dev-engg File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I don’t fully understand why is_extmu is excluded here. Couldn’t the KAT hash for externalMu also be defined in the upstream YAML file?
Uh oh!
There was an error while loading. Please reload this page.
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.
Hi @bhess ,
I disabled KAT tests for mu variants as kat_sig.c tests varying message lengths, but extmu strictly requires exactly 64 bytes for the message.
Since NIST KATs expect varying lengths, I disabled them as msglen is constant here.
how would you prefer to handle this for external-mu APIs?
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.
Ok, I suppose known-answer tests for externalMu are covered by the ACVP tests?
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.
Yes, correct. ACVP tests cover external-mu