-
Notifications
You must be signed in to change notification settings - Fork 562
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
SPHINCS+ #3533
SPHINCS+ #3533
Conversation
I noticed you defined params for Haraka as well. If you plan to support this (I am ambivalent either way on this point) please create a distinct PR for adding it. |
It's not quite clear whether we'll add Haraka. Currently the parameters were added for completeness. Certainly, support for Haraka won't come in this first PR. |
- add mgf1 and shake as required for sphincsplus Later at least shake should probably be optional - remove thread_pool.h import - update comment
Co-Authored-By: Amos Treiber<amos.treiber@rohde-schwarz.com> Co-Authored-By: Rene Meusel<rene.meusel@rohde-schwarz.com>
auto sha256 = [](const Botan::Sphincs_Address& adrs) | ||
{ | ||
auto h = Botan::HashFunction::create_or_throw("SHA-256"); | ||
adrs.apply_to_hash(*h); | ||
return h->final_stdvec(); | ||
}; |
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.
Add test for apply_to_hash_compressed()
if(result.tests_failed() > 0){ | ||
int x = 0; // Dummy | ||
} |
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.
?
Closing in favor of #3549. |
#3099
This is a huge mess at the moment... no need to look deeper into it at this point.
TODO
Implementation Milestones
That's a rough collection of implementation work packages. We'll keep it updated as things progress.