commit | efb0e16ee5fbe64dd67ba4b84c3c24dc55fa44c6 | [log] [tgz] |
---|---|---|
author | Adam Langley <agl@google.com> | Thu Jul 09 11:35:04 2015 -0700 |
committer | Adam Langley <agl@google.com> | Thu Jul 09 22:47:14 2015 +0000 |
tree | bdb3688cd8a7b6364ff9659b9d3c3773bd948744 | |
parent | d822ed811a1242c18ea7b16a905f0abdb2061f66 [diff] [blame] |
Reject empty ALPN protocols. https://tools.ietf.org/html/rfc7301#section-3.1 specifies that a ProtocolName may not be empty. This change enforces this in ClientHello and ServerHello messages. Thanks to Doug Hogan for reporting this. Change-Id: Iab879c83145007799b94d2725201ede1a39e4596 Reviewed-on: https://boringssl-review.googlesource.com/5390 Reviewed-by: Adam Langley <agl@google.com>
diff --git a/ssl/test/runner/common.go b/ssl/test/runner/common.go index 2e30208..bad3ebe 100644 --- a/ssl/test/runner/common.go +++ b/ssl/test/runner/common.go
@@ -544,6 +544,10 @@ // of ALPN works regardless of their relative order. SwapNPNAndALPN bool + // ALPNProtocol, if not nil, sets the ALPN protocol that a server will + // return. + ALPNProtocol *string + // AllowSessionVersionMismatch causes the server to resume sessions // regardless of the version associated with the session. AllowSessionVersionMismatch bool