tree 7de374ed19b44ec3c6937d1c0266876a23bd31d4
parent fa214e4a18785a4281375b56c00732b470e9cd5b
author David Benjamin <davidben@google.com> 1462917452 -0400
committer David Benjamin <davidben@google.com> 1463520463 +0000

Condition the read_close_notify check on type, not shutdown state.

The logic to drop records really should be in the caller. Unless
ssl3_read_bytes is broken apart, condition on the type field which is more
robust.

If we manage to call, say, SSL_read after SSL_shutdown completes at 0 (instead
of 1), this logic can incorrectly cause unknown record types to be dropped.

Change-Id: Iab90e5d9190fcccbf6ff55e17079a2704ed99901
Reviewed-on: https://boringssl-review.googlesource.com/7953
Reviewed-by: Steven Valdez <svaldez@google.com>
Reviewed-by: David Benjamin <davidben@google.com>