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

xtest: pkcs11_1021: skip CKM_RSA_PKCS test is not supported #763

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

etienne-lms
Copy link
Contributor

Skip CKM_RSA_PKCS test in pkcs11_1021 is the mechanism is not supported. This is likely to append when OP-TEE core is built with CFG_CRYPTO_RSASSA_NA1 disabled as when built with MbedTLS as core crypto library.

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.

With s/append/happen/:

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

@jforissier
Copy link
Contributor

...and "s/is not supported/if not supported/"

Skip CKM_RSA_PKCS test in pkcs11_1021 if the mechanism is not supported.
This is likely to happen when OP-TEE core is built with
CFG_CRYPTO_RSASSA_NA1 disabled as when built with MbedTLS as core crypto
library.

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

Commit message fixed and review tag applied.

@jforissier jforissier merged commit f75bb37 into OP-TEE:master Nov 26, 2024
1 check passed
@etienne-lms etienne-lms deleted the pkcs11-1021 branch December 12, 2024 17:19
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