[openssl-commits] [openssl] OpenSSL_1_1_0-stable update

[Richard Levitte]

The branch OpenSSL_1_1_0-stable has been updated
       via  fbd3e06b64a1d4733eb95fc3ed0e35bb4c3a726e (commit)
      from  64903a26c5855347738825d7724e76e8a89180f3 (commit)


- Log -----------------------------------------------------------------
commit fbd3e06b64a1d4733eb95fc3ed0e35bb4c3a726e
Author: Richard Levitte <levi...@openssl.org>
Date:   Sat Jul 8 22:13:24 2017 +0200

    Fix cipher_compare
    
    Unsigned overflow.  Found by Brian Carpenter
    
    Fixes #3889
    
    Reviewed-by: Tim Hudson <t...@openssl.org>
    (Merged from https://github.com/openssl/openssl/pull/3890)
    
    (cherry picked from commit a7ff57965b81ce4fd73a18266ce29abf6b909fdb)

-----------------------------------------------------------------------

Summary of changes:
 ssl/s3_lib.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/ssl/s3_lib.c b/ssl/s3_lib.c
index d45a246..9ea02dc 100644
--- a/ssl/s3_lib.c
+++ b/ssl/s3_lib.c
@@ -2725,7 +2725,9 @@ static int cipher_compare(const void *a, const void *b)
     const SSL_CIPHER *ap = (const SSL_CIPHER *)a;
     const SSL_CIPHER *bp = (const SSL_CIPHER *)b;
 
-    return ap->id - bp->id;
+    if (ap->id == bp->id)
+        return 0;
+    return ap->id < bp->id ? -1 : 1;
 }
 
 void ssl_sort_cipher_list(void)
_____
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits