Skip to content
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

Refactor ED25519_keypair into hw and nohw backend #1271

Merged

Conversation

torben-hansen
Copy link
Contributor

@torben-hansen torben-hansen commented Oct 27, 2023

Description of changes:

Refactors ED25519_keypair into a no-hardware and a hardware path. Existing backend is the former. Preparation for s2n-bignum ED25519.

s2n-bignum is lacking SHA512. So unfortunately, I can't punt this computation to s2n-bignum.

PR chain:

  1. (this PR) refactor ED25519_keypair
  2. refactor ED25519_sign
  3. refactor ED25519_verify
  4. various small changes to improve code quality
  5. import s2n-bignum Ed25519 routines
  6. connect s2n-bignum Ed25519 backend to frontend

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and the ISC license.

@torben-hansen torben-hansen requested a review from a team as a code owner October 27, 2023 23:31
@torben-hansen torben-hansen changed the title Refactor ED25519_sign into hw and nohw backend Refactor ED25519_keypair into hw and nohw backend Oct 27, 2023
@torben-hansen torben-hansen enabled auto-merge (squash) October 27, 2023 23:32
@torben-hansen torben-hansen force-pushed the refactor_ed25519_to_prep_for_s2n_bignum branch from d3e395c to c47dc7f Compare October 28, 2023 19:02
@torben-hansen torben-hansen requested a review from nebeid October 30, 2023 15:03
dkostic
dkostic previously approved these changes Oct 30, 2023
crypto/curve25519/curve25519.c Outdated Show resolved Hide resolved
@torben-hansen torben-hansen requested a review from nebeid November 1, 2023 14:52
@torben-hansen torben-hansen merged commit 6b32360 into aws:main Nov 1, 2023
11 checks passed
@justsmth justsmth mentioned this pull request Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants