From dc415d9ff18dbde023f8518505fec323881e566a Mon Sep 17 00:00:00 2001 From: Eugene Syromiatnikov Date: Thu, 14 Aug 2025 09:58:01 +0200 Subject: [PATCH] test/bioprinttest.c: use the whole buffer for checks MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit There is no need to decrement the size of buffer by 1, snprintf should terminate the output with a \0. Signed-off-by: Eugene Syromiatnikov Reviewed-by: Saša Nedvědický Reviewed-by: Neil Horman (Merged from https://github.com/openssl/openssl/pull/28177) --- test/bioprinttest.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/bioprinttest.c b/test/bioprinttest.c index 27dcc517ac1..a9dcdb02c9b 100644 --- a/test/bioprinttest.c +++ b/test/bioprinttest.c @@ -141,7 +141,7 @@ static int test_zu(int i) memset(bio_buf, '@', sizeof(bio_buf)); - BIO_snprintf(bio_buf, sizeof(bio_buf) - 1, data->format, data->value); + BIO_snprintf(bio_buf, sizeof(bio_buf), data->format, data->value); if (!TEST_str_eq(bio_buf, data->expected)) return 0; return 1; @@ -171,7 +171,7 @@ static int test_j(int i) memset(bio_buf, '@', sizeof(bio_buf)); - BIO_snprintf(bio_buf, sizeof(bio_buf) - 1, data->format, data->value); + BIO_snprintf(bio_buf, sizeof(bio_buf), data->format, data->value); if (!TEST_str_eq(bio_buf, data->expected)) return 0; return 1; -- 2.47.3