From: Eugene Syromiatnikov Date: Thu, 14 Aug 2025 07:58:01 +0000 (+0200) Subject: test/bioprinttest.c: use the whole buffer for checks X-Git-Tag: openssl-3.6.0-alpha1~29 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=dc415d9ff18dbde023f8518505fec323881e566a;p=thirdparty%2Fopenssl.git test/bioprinttest.c: use the whole buffer for checks 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) --- 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;