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

[Matt Caswell]

The branch OpenSSL_1_1_0-stable has been updated
       via  5bfb357a0d2046fc75daf67a5bc019eb87443729 (commit)
      from  b2621ffdf8063c961ee5c3a7dd9db68d623d3554 (commit)


- Log -----------------------------------------------------------------
commit 5bfb357a0d2046fc75daf67a5bc019eb87443729
Author: Matt Caswell <matt at openssl.org>
Date:   Sun Dec 10 09:55:08 2017 +0000

    Fix a switch statement fallthrough
    
    SSL_trace() has a case which was inadvertently falling through.
    
    Reviewed-by: Richard Levitte <levitte at openssl.org>
    Reviewed-by: Rich Salz <rsalz at openssl.org>
    (Merged from https://github.com/openssl/openssl/pull/4888)

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

Summary of changes:
 ssl/t1_trce.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/ssl/t1_trce.c b/ssl/t1_trce.c
index 3bb9b6d..76bdf79 100644
--- a/ssl/t1_trce.c
+++ b/ssl/t1_trce.c
@@ -1301,13 +1301,15 @@ void SSL_trace(int write_p, int version, int content_type,
         break;
 
     case SSL3_RT_ALERT:
-        if (msglen != 2)
+        if (msglen != 2) {
             BIO_puts(bio, "    Illegal Alert Length\n");
-        else {
+        } else {
             BIO_printf(bio, "    Level=%s(%d), description=%s(%d)\n",
                        SSL_alert_type_string_long(msg[0] << 8),
                        msg[0], SSL_alert_desc_string_long(msg[1]), msg[1]);
         }
+        break;
+
     case DTLS1_RT_HEARTBEAT:
         ssl_print_heartbeat(bio, 4, msg, msglen);
         break;