commit ee0b02a10d3669cf33cef30ec380a1bc69e8384d
author David Benjamin <davidben@chromium.org> Sat Aug 22 11:35:40 2015 -0400
committer Adam Langley <agl@google.com> Mon Aug 31 19:08:24 2015 +0000
tree 2046134ae90d4e494655b8931697ad39861e526c
parent e9cb2ec832bc3225f617551c5bb2ba9170c74abc

Don't confuse TLS bidirectional shutdown on record type zero.

The bidi shutdown code uses type = 0 as a special signal value, but code
elsewhere doesn't account for this.

BUG=526437

Change-Id: I090cee421633d70ef3b84f4daa811608031b9ed9
Reviewed-on: https://boringssl-review.googlesource.com/5771
Reviewed-by: Adam Langley <agl@google.com>

ssl/s3_pkt.c

diff --git a/ssl/s3_pkt.c b/ssl/s3_pkt.c
index 250973e..50898ed 100644
--- a/ssl/s3_pkt.c
+++ b/ssl/s3_pkt.c
@@ -458,7 +458,7 @@
     return 0;
   }
 
-  if (type == rr->type) {
+  if (type != 0 && type == rr->type) {
     s->s3->warning_alert_count = 0;
 
     /* SSL3_RT_APPLICATION_DATA or SSL3_RT_HANDSHAKE */