From 5bfb357a0d2046fc75daf67a5bc019eb87443729 Mon Sep 17 00:00:00 2001 From: Matt Caswell Date: Sun, 10 Dec 2017 09:55:08 +0000 Subject: [PATCH] Fix a switch statement fallthrough SSL_trace() has a case which was inadvertently falling through. Reviewed-by: Richard Levitte Reviewed-by: Rich Salz (Merged from https://github.com/openssl/openssl/pull/4888) --- 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 3bb9b6d7e5..76bdf792ae 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; -- 2.25.1