Enable all keys for interruptible op fail tests
Due to a misunderstanding about the purpose of the test, I had limited this to ECC keys only, however this defeats the purpose of the test, and left gaps in test coverage. Signed-off-by: Paul Elliott <paul.elliott@arm.com>
This commit is contained in:
parent
57897b8d6a
commit
ddbc6ed6cd
1 changed files with 9 additions and 13 deletions
|
@ -245,13 +245,11 @@ void sign_fail(int key_type_arg, data_t *key_data,
|
||||||
input, sizeof(input),
|
input, sizeof(input),
|
||||||
output, sizeof(output), &length));
|
output, sizeof(output), &length));
|
||||||
|
|
||||||
if (PSA_KEY_TYPE_IS_ECC(key_type)) {
|
TEST_STATUS(expected_status,
|
||||||
TEST_STATUS(expected_status,
|
psa_sign_hash_start(&sign_operation, key_id, alg,
|
||||||
psa_sign_hash_start(&sign_operation, key_id, alg,
|
input, sizeof(input)));
|
||||||
input, sizeof(input)));
|
|
||||||
|
|
||||||
PSA_ASSERT(psa_sign_hash_abort(&sign_operation));
|
PSA_ASSERT(psa_sign_hash_abort(&sign_operation));
|
||||||
}
|
|
||||||
|
|
||||||
if (!private_only) {
|
if (!private_only) {
|
||||||
/* Determine a plausible signature size to avoid an INVALID_SIGNATURE
|
/* Determine a plausible signature size to avoid an INVALID_SIGNATURE
|
||||||
|
@ -270,14 +268,12 @@ void sign_fail(int key_type_arg, data_t *key_data,
|
||||||
input, sizeof(input),
|
input, sizeof(input),
|
||||||
output, output_length));
|
output, output_length));
|
||||||
|
|
||||||
if (PSA_KEY_TYPE_IS_ECC(key_type)) {
|
TEST_STATUS(expected_status,
|
||||||
TEST_STATUS(expected_status,
|
psa_verify_hash_start(&verify_operation, key_id, alg,
|
||||||
psa_verify_hash_start(&verify_operation, key_id, alg,
|
input, sizeof(input),
|
||||||
input, sizeof(input),
|
output, output_length));
|
||||||
output, output_length));
|
|
||||||
|
|
||||||
PSA_ASSERT(psa_verify_hash_abort(&verify_operation));
|
PSA_ASSERT(psa_verify_hash_abort(&verify_operation));
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
exit:
|
exit:
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue