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

sys/psa_crypto: return proper error code for psa_verify_msg #20346

Merged
merged 1 commit into from
Feb 6, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion pkg/c25519/psa_c25519/edsign.c
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ psa_status_t psa_ecc_ed25519_verify_message(const uint8_t *key_buffer,

ret = edsign_verify(signature, key_buffer, input, input_length);
if (!ret) {
return PSA_ERROR_GENERIC_ERROR;
return PSA_ERROR_INVALID_SIGNATURE;
}

(void)key_buffer_size;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,7 @@
case CRYS_ECMONT_PKI_ERROR:
case CRYS_ECMONT_IS_NOT_SUPPORTED:
case CRYS_ECEDW_IS_NOT_SUPPORTED:
return PSA_ERROR_INVALID_ARGUMENT;
return PSA_ERROR_INVALID_ARGUMENT;
default:
return PSA_ERROR_GENERIC_ERROR;
}
Expand Down Expand Up @@ -182,7 +182,7 @@

const char *cryptocell310_status_to_humanly_readable(uint32_t status)
{
switch(status) {

Check warning on line 185 in pkg/driver_cryptocell_310/psa_cryptocell_310/error_conversion.c

View workflow job for this annotation

GitHub Actions / static-tests

keyword 'switch' not followed by a single space
case CRYS_ECDH_SVDP_DH_NOT_CONCENT_PUBL_AND_PRIV_DOMAIN_ID_ERROR:
return "CRYS_ECDH_SVDP_DH_NOT_CONCENT_PUBL_AND_PRIV_DOMAIN_ID_ERROR";
case CRYS_ECDSA_SIGN_INVALID_IS_EPHEMER_KEY_INTERNAL_ERROR:
Expand Down
Loading