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

libckteec: support for CKM_RSA_X_509 #386

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

etienne-lms
Copy link
Contributor

Define CKM_RSA_X_509 mechanism identifier to allow client to request operations with this mechanism.

Related to OP-TEE/optee_os#7030.

Copy link

github-actions bot commented Oct 7, 2024

This pull request has been marked as a stale pull request because it has been open (more than) 30 days with no activity. Remove the stale label or add a comment, otherwise this pull request will automatically be closed in 5 days. Note, that you can always re-open a closed issue at any time.

@github-actions github-actions bot added the Stale label Oct 7, 2024
@jforissier jforissier removed the Stale label Oct 7, 2024
Copy link
Contributor

@jforissier jforissier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Acked-by: Jerome Forissier <[email protected]>

Define CKM_RSA_X_509 mechanism identifier to allow client to
request operations with this mechanism.

Signed-off-by: Alexandre Marechal <[email protected]>
Signed-off-by: Etienne Carriere <[email protected]>
Acked-by: Jerome Forissier <[email protected]>
@etienne-lms
Copy link
Contributor Author

Tag applied

Copy link

This pull request has been marked as a stale pull request because it has been open (more than) 30 days with no activity. Remove the stale label or add a comment, otherwise this pull request will automatically be closed in 5 days. Note, that you can always re-open a closed issue at any time.

@etienne-lms
Copy link
Contributor Author

@jforissier, this change is harmless and at least makes libckteec ready to support CKM_RSA_X_509 PKCS#11 mechanism, what ever it is supported or not by the pkcs11 TA. Would you be OK to merge it, regardless of the state and destiny of P-R OP-TEE/optee_os#7030?

@jforissier
Copy link
Contributor

@etienne-lms fine with me. Hopefully we can get the rest of the code merged at some point, since everything is ready including the tests.

@jforissier jforissier merged commit 54539c5 into OP-TEE:master Nov 18, 2024
2 checks passed
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.

2 participants