Add safety for keys larger than we currently support.
Prevent buffer overflow with keys whos grp.nbits is greater than PSA_VENDOR_ECC_MAX_CURVE_BITS. Signed-off-by: Paul Elliott <paul.elliott@arm.com>
This commit is contained in:
parent
93d9ca83ea
commit
ba70ad4944
1 changed files with 10 additions and 0 deletions
|
@ -3577,6 +3577,11 @@ psa_status_t mbedtls_psa_sign_hash_start(
|
|||
required_hash_length = (hash_length < operation->coordinate_bytes ?
|
||||
hash_length : operation->coordinate_bytes);
|
||||
|
||||
if (required_hash_length > sizeof(operation->hash)) {
|
||||
/* Shouldn't happen, but better safe than sorry. */
|
||||
return PSA_ERROR_CORRUPTION_DETECTED;
|
||||
}
|
||||
|
||||
memcpy(operation->hash, hash, required_hash_length);
|
||||
operation->hash_length = required_hash_length;
|
||||
|
||||
|
@ -3812,6 +3817,11 @@ psa_status_t mbedtls_psa_verify_hash_start(
|
|||
required_hash_length = (hash_length < coordinate_bytes ? hash_length :
|
||||
coordinate_bytes);
|
||||
|
||||
if (required_hash_length > sizeof(operation->hash)) {
|
||||
/* Shouldn't happen, but better safe than sorry. */
|
||||
return PSA_ERROR_CORRUPTION_DETECTED;
|
||||
}
|
||||
|
||||
memcpy(operation->hash, hash, required_hash_length);
|
||||
operation->hash_length = required_hash_length;
|
||||
|
||||
|
|
Loading…
Reference in a new issue