diff options
| author | Janne Peltonen <janne.peltonen@nokia.com> | 2022-02-18 13:54:40 +0200 |
|---|---|---|
| committer | Petri Savolainen <petri.savolainen@nokia.com> | 2022-02-23 15:21:35 +0200 |
| commit | c496ecaa8daf78a65c4fad71ada169c79754c8f5 (patch) | |
| tree | 94a4dc9e512bb2a09e3bca09fb2d6ef34bb378d1 /test/validation | |
| parent | b86015db74005cd1e040b6728fb38c087919e1a0 (diff) | |
validation: crypto: fix checking of AAD length capability
Checking of AAD length capability is entirely missing. Add a check that
the AAD length used in a test vector is actually supported before
executing the test.
Signed-off-by: Janne Peltonen <janne.peltonen@nokia.com>
Reviewed-by: Jere Leppänen <jere.leppanen@nokia.com>
Diffstat (limited to 'test/validation')
| -rw-r--r-- | test/validation/api/crypto/odp_crypto_test_inp.c | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/test/validation/api/crypto/odp_crypto_test_inp.c b/test/validation/api/crypto/odp_crypto_test_inp.c index ecfc806b0..53ae5c37d 100644 --- a/test/validation/api/crypto/odp_crypto_test_inp.c +++ b/test/validation/api/crypto/odp_crypto_test_inp.c @@ -777,6 +777,19 @@ static void alg_test(odp_crypto_op_t op, CU_ASSERT(!rc); } +static odp_bool_t aad_len_ok(const odp_crypto_auth_capability_t *capa, uint32_t len) +{ + if (len < capa->aad_len.min || len > capa->aad_len.max) + return false; + + if (len == capa->aad_len.min) + return true; + if (capa->aad_len.inc == 0) + return false; + + return ((len - capa->aad_len.min) % capa->aad_len.inc) == 0; +} + static void check_alg(odp_crypto_op_t op, odp_cipher_alg_t cipher_alg, odp_auth_alg_t auth_alg, @@ -846,6 +859,7 @@ static void check_alg(odp_crypto_op_t op, ref[idx].auth_iv_length && auth_capa[i].key_len == ref[idx].auth_key_length && + aad_len_ok(&auth_capa[i], ref[idx].aad_length) && auth_capa[i].bit_mode == bit_mode) { auth_idx = i; |