commit af9513bb48407544c301fe2a06b69793c38e7a8a
author makise-homura <akemi_homura@kurisa.ch> Mon Aug 24 18:26:27 2020 +0300
committer makise-homura <akemi_homura@kurisa.ch> Mon Aug 24 23:42:49 2020 +0300
tree 5e9bd76a0f23ca9942288cecb426637800ff6d15
parent e014fece509a5c486c685f384e1286fba2653ab7

A different approach of signed-to-unsigned comparison

Suggsted by @hanno-arm

Signed-off-by: makise-homura <akemi_homura@kurisa.ch>

library/ssl_msg.c

diff --git a/library/ssl_msg.c b/library/ssl_msg.c
index 259a71d..0d74e6d 100644
--- a/library/ssl_msg.c
+++ b/library/ssl_msg.c
@@ -2048,7 +2048,7 @@
             if( ret < 0 )
                 return( ret );
 
-            if ( (size_t)ret > len || ( INT_MAX > SIZE_MAX && (size_t)ret > SIZE_MAX ) )
+            if ( (size_t)ret > len || ( INT_MAX > SIZE_MAX && ret > (int)SIZE_MAX ) )
             {
                 MBEDTLS_SSL_DEBUG_MSG( 1,
                     ( "f_recv returned %d bytes but only %lu were requested",
@@ -2102,7 +2102,7 @@
         if( ret <= 0 )
             return( ret );
 
-        if( (size_t)ret > ssl->out_left || ( INT_MAX > SIZE_MAX && (size_t)ret > SIZE_MAX ) )
+        if( (size_t)ret > ssl->out_left || ( INT_MAX > SIZE_MAX && ret > (int)SIZE_MAX ) )
         {
             MBEDTLS_SSL_DEBUG_MSG( 1,
                 ( "f_send returned %d bytes but only %lu bytes were sent",