Skip to content

Commit

Permalink
Merge pull request #102 from bergzand/pr/test/mbedtls_skip_length_com…
Browse files Browse the repository at this point in the history
…pare

tests: Skip length compare for ecdsa
  • Loading branch information
bergzand authored Apr 6, 2021
2 parents 4ed4b15 + 127f7c9 commit 5e04348
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions tests/sign.c
Original file line number Diff line number Diff line change
Expand Up @@ -433,6 +433,7 @@ void test_sign8(void)
cose_sign_enc_t sign;
cose_signature_t signature;
cose_key_t key;

/* Initialize struct */
cose_sign_init(&sign, 0);
cose_signature_init(&signature);
Expand All @@ -448,15 +449,18 @@ void test_sign8(void)
/* Octet stream content type */

/* Encode COSE sign object */
size_t size_no_aad = cose_sign_encode(&sign, buf, sizeof(buf), &psign);
ssize_t size_no_aad = cose_sign_encode(&sign, buf, sizeof(buf), &psign);

CU_ASSERT_NOT_EQUAL_FATAL(size_no_aad, 0);

cose_sign_set_external_aad(&sign, sign_aad, strlen(sign_aad));

size_t size_w_aad = cose_sign_encode(&sign, buf, sizeof(buf), &psign);
ssize_t size_w_aad = cose_sign_encode(&sign, buf, sizeof(buf), &psign);

CU_ASSERT_EQUAL(size_w_aad, size_no_aad);
if (key.algo != COSE_ALGO_ES512) {
/* ECDSA signatures have variable lengths, no sense in checking this */
CU_ASSERT_EQUAL(size_w_aad, size_no_aad);
}

cose_sign_dec_t verify;
cose_signature_dec_t vsignature;
Expand Down

0 comments on commit 5e04348

Please sign in to comment.