commit 329fe7e0438f390815a01d127aabe37dd2eca525
author makise-homura <akemi_homura@kurisa.ch> Mon Aug 24 18:39:56 2020 +0300
committer makise-homura <akemi_homura@kurisa.ch> Mon Aug 24 18:39:56 2020 +0300
tree 6a17395c0e83faf919f72d5006bd8a620b0e437b
parent 03c2b8f1c793cbbf7cadc1c0606a040f7ed7f0fd

A different approach of signed-to-unsigned comparison

Suggested by @hanno-arm

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

library/ssl_tls.c

diff --git a/library/ssl_tls.c b/library/ssl_tls.c
index 7f2170a..76fbd3b 100644
--- a/library/ssl_tls.c
+++ b/library/ssl_tls.c
@@ -2581,7 +2581,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",
@@ -2636,7 +2636,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",