commit 740b218386083dc708ce98ccc94a63a95cd5629e
author Krzysztof Stachowiak <krzysiek.stachowiak@gmail.com> Tue Mar 13 11:31:14 2018 +0100
committer Krzysztof Stachowiak <krzysiek.stachowiak@gmail.com> Tue Mar 13 11:31:14 2018 +0100
tree 8307679eb2027e3c417d3bbfacd9f5be683a6da1
parent 469b8829479f031172e24eedfd473eac0782830c

Add bounds check before length read

library/ssl_cli.c

diff --git a/library/ssl_cli.c b/library/ssl_cli.c
index 2534346..585750e 100644
--- a/library/ssl_cli.c
+++ b/library/ssl_cli.c
@@ -2057,6 +2057,12 @@
      *
      * opaque psk_identity_hint<0..2^16-1>;
      */
+    if( (*p) > end - 2 )
+    {
+        MBEDTLS_SSL_DEBUG_MSG( 1, ( "bad server key exchange message "
+                                    "(psk_identity_hint length)" ) );
+        return( MBEDTLS_ERR_SSL_BAD_HS_SERVER_KEY_EXCHANGE );
+    }
     len = (*p)[0] << 8 | (*p)[1];
     *p += 2;