diff options
| author | Matt Caswell <matt@openssl.org> | 2017-12-10 09:55:08 +0000 |
|---|---|---|
| committer | Matt Caswell <matt@openssl.org> | 2017-12-11 09:46:59 +0000 |
| commit | 5bfb357a0d2046fc75daf67a5bc019eb87443729 (patch) | |
| tree | b645815d4f01c2defdcac19ffc1a09e863a5b113 /ssl | |
| parent | b2621ffdf8063c961ee5c3a7dd9db68d623d3554 (diff) | |
Fix a switch statement fallthrough
SSL_trace() has a case which was inadvertently falling through.
Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/4888)
Diffstat (limited to 'ssl')
| -rw-r--r-- | ssl/t1_trce.c | 6 |
1 files 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; |