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

Support for the MAXQ1065 wolfPKCS11 effort. #8179

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

anhu
Copy link
Member

@anhu anhu commented Nov 13, 2024

  • in wc_ecc_import_private_key_ex, check to make sure devid is not invalid before calling wc_MAXQ10XX_EccSetKey. This is because we sometimes need to locally sign material.
  • in various places, delete a key in case it already exists; ignore error since it might not exist.
  • in various places, unlock and then lock the HW mutex around ECDSA_sign() because it needs access to rng
  • in wolfSSL_MAXQ10XX_CryptoDevCb allow 1065 to call the crypto callback.
  • in wolfSSL_MAXQ10XX_CryptoDevCb don't set the key during signing; use pre-provisioned one instead; DEVICE_KEY_PAIR_OBJ_ID
  • add ecc_establish and modifications to key creation to support ECDH
  • support AES CBC

- in wc_ecc_import_private_key_ex, check to make sure devid is not invalid
  before calling wc_MAXQ10XX_EccSetKey. This is because we sometimes need to
  locally sign material.
- in various places, delete a key in case it already exists; ignore error since
  it might not exist.
- in various places, unlock and then lock the HW mutex around ECDSA_sign()
  because it needs access to rng
- in wolfSSL_MAXQ10XX_CryptoDevCb allow 1065 to call the crypto callback.
- in wolfSSL_MAXQ10XX_CryptoDevCb don't set the key during signing; use
  pre-provisioned one instead; DEVICE_KEY_PAIR_OBJ_ID
- add ecc_establish and modifications to key creation to support ECDH
- support AES CBC
@anhu anhu self-assigned this Nov 13, 2024
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.

1 participant