From 8df10232cf7bc6b82cbc3e19429fb551d36da97b Mon Sep 17 00:00:00 2001 From: Hanno Becker Date: Wed, 10 Oct 2018 15:48:39 +0100 Subject: [PATCH] Add explicit unsigned-to-signed integer conversion The previous code triggered a compiler warning because of a comparison of a signed and an unsigned integer. The conversion is safe because `len` is representable by 16-bits, hence smaller than the maximum integer. --- library/ssl_cli.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/library/ssl_cli.c b/library/ssl_cli.c index b15bc515e..74883b3c3 100644 --- a/library/ssl_cli.c +++ b/library/ssl_cli.c @@ -2106,7 +2106,7 @@ static int ssl_parse_server_psk_hint( mbedtls_ssl_context *ssl, len = (*p)[0] << 8 | (*p)[1]; *p += 2; - if( end - (*p) < len ) + if( end - (*p) < (int) len ) { MBEDTLS_SSL_DEBUG_MSG( 1, ( "bad server key exchange message " "(psk_identity_hint length)" ) );