diff options
author | Pauli <pauli@openssl.org> | 2022-11-02 12:20:16 +1100 |
---|---|---|
committer | Tomas Mraz <tomas@openssl.org> | 2022-11-03 10:31:29 +0100 |
commit | ce0a7cadadb973216399e70d3a69f352b0843deb (patch) | |
tree | ffdfa6a4be911bf563b45e15269d337015c7175c /test/punycode_test.c | |
parent | 00e38edcfb95b556a59de96e0c18343828929c8f (diff) | |
download | openssl-ce0a7cadadb973216399e70d3a69f352b0843deb.zip openssl-ce0a7cadadb973216399e70d3a69f352b0843deb.tar.gz openssl-ce0a7cadadb973216399e70d3a69f352b0843deb.tar.bz2 |
Coverity 1516624: Fix overrun memory access.
Not possible to hit but good to address.
Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19576)
Diffstat (limited to 'test/punycode_test.c')
-rw-r--r-- | test/punycode_test.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/punycode_test.c b/test/punycode_test.c index 113d04b..9d81713 100644 --- a/test/punycode_test.c +++ b/test/punycode_test.c @@ -154,7 +154,7 @@ static int test_punycode(int n) if (!TEST_true(ossl_punycode_decode(tc->encoded, strlen(tc->encoded), buffer, &bsize))) return 0; - for (i = 0; i < sizeof(tc->raw); i++) + for (i = 0; i < OSSL_NELEM(tc->raw); i++) if (tc->raw[i] == 0) break; if (!TEST_mem_eq(buffer, bsize * sizeof(*buffer), |