From 34018bef3dcc8172a7b94f92939c55a79d6656f4 Mon Sep 17 00:00:00 2001 From: Rodrigo Dias Correa Date: Tue, 10 Nov 2020 02:51:51 -0300 Subject: [PATCH] Fix GCC warning in `ssl_calc_finished_tls_sha384` GCC 11 generated a warning because `padbuf` was too small to be used as an argument for `mbedtls_sha512_finish_ret`. The `output` parameter of `mbedtls_sha512_finish_ret` has the type `unsigned char[64]`, but `padbuf` was only 48 bytes long. Even though `ssl_calc_finished_tls_sha384` uses only 48 bytes for the hash output, the size of `padbuf` was increased to 64 bytes. Signed-off-by: Rodrigo Dias Correa --- library/ssl_tls.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/library/ssl_tls.c b/library/ssl_tls.c index 725e9e7ef..8757e4659 100644 --- a/library/ssl_tls.c +++ b/library/ssl_tls.c @@ -5411,7 +5411,7 @@ static void ssl_calc_finished_tls_sha384( int len = 12; const char *sender; mbedtls_sha512_context sha512; - unsigned char padbuf[48]; + unsigned char padbuf[64]; mbedtls_ssl_session *session = ssl->session_negotiate; if( !session )