commit | 8f299d5e03cb11b9f6f1b9dbc6f5b1c1033e4424 | [log] [tgz] |
---|---|---|
author | Matthew Braithwaite <mab@google.com> | Mon Feb 10 16:15:37 2020 -0800 |
committer | CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> | Tue Feb 11 01:14:36 2020 +0000 |
tree | 19a2a5c5f20d6857b3ee9fafaaec6035b781d1b6 | |
parent | 4f3e8212ea413d0bc271054a12ed581bc742e825 [diff] |
SSL_apply_handback: don't choke on trailing data. It may be useful for future extensibility. Change-Id: I415095140367a44a2c8dd636998721399232c400 Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/39964 Reviewed-by: David Benjamin <davidben@google.com> Commit-Queue: David Benjamin <davidben@google.com>
diff --git a/ssl/handoff.cc b/ssl/handoff.cc index abe996b..3926939 100644 --- a/ssl/handoff.cc +++ b/ssl/handoff.cc
@@ -671,8 +671,7 @@ (hs->key_shares[0] = SSLKeyShare::Create(&key_share)) == nullptr) { return false; } - - return CBS_len(&seq) == 0; + return true; // Trailing data allowed for extensibility. } BSSL_NAMESPACE_END