diff options
author | Reinhard Urban <reinhard.urban@nubix.de> | 2022-10-28 09:56:05 +0200 |
---|---|---|
committer | Tomas Mraz <tomas@openssl.org> | 2022-11-07 19:21:56 +0100 |
commit | 91b7520e2385a513ad879dfa8fe8e45466315a27 (patch) | |
tree | 2ade15beb717934306a5161ad8ae23627e843078 /test | |
parent | 6db5cb844852f8f13753caf26dea7750f701e4d9 (diff) |
Fix PACKET_equal test with BUF_LEN+1 on -Wstringop-overread
Either suppress the error, or better make smbuf longer.
Detected with -Werror.
CLA: trivial
Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19528)
Diffstat (limited to 'test')
-rw-r--r-- | test/packettest.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/test/packettest.c b/test/packettest.c index c62247f9be..68b6d679b9 100644 --- a/test/packettest.c +++ b/test/packettest.c @@ -12,13 +12,13 @@ #define BUF_LEN 255 -static unsigned char smbuf[BUF_LEN]; +static unsigned char smbuf[BUF_LEN + 1]; static int test_PACKET_remaining(void) { PACKET pkt; - if (!TEST_true(PACKET_buf_init(&pkt, smbuf, sizeof(smbuf))) + if (!TEST_true(PACKET_buf_init(&pkt, smbuf, BUF_LEN)) || !TEST_size_t_eq(PACKET_remaining(&pkt), BUF_LEN) || !TEST_true(PACKET_forward(&pkt, BUF_LEN - 1)) || !TEST_size_t_eq(PACKET_remaining(&pkt), 1) @@ -33,7 +33,7 @@ static int test_PACKET_end(void) { PACKET pkt; - if (!TEST_true(PACKET_buf_init(&pkt, smbuf, sizeof(smbuf))) + if (!TEST_true(PACKET_buf_init(&pkt, smbuf, BUF_LEN)) || !TEST_size_t_eq(PACKET_remaining(&pkt), BUF_LEN) || !TEST_ptr_eq(PACKET_end(&pkt), smbuf + BUF_LEN) || !TEST_true(PACKET_forward(&pkt, BUF_LEN - 1)) |