Satisfy golint.

Errors are supposed to be fragments that go into sentences, rather than
sentences themselves.

Change-Id: I6569fce25535475162c85e7b0db7eeb62c93febd
Reviewed-on: https://boringssl-review.googlesource.com/c/33324
Commit-Queue: Adam Langley <agl@google.com>
Reviewed-by: Adam Langley <agl@google.com>
This commit is contained in:
David Benjamin 2018-11-21 14:34:51 -06:00 committed by CQ bot account: commit-bot@chromium.org
parent bbc429148f
commit 9113e0996f

View File

@ -59,12 +59,15 @@ func fixClientHellos(hello *clientHelloMsg, in []byte) ([]byte, error) {
hello.random = newHello.random
hello.sessionId = newHello.sessionId
// Replace |ret|'s key shares with those of |hello|.
// Replace |ret|'s key shares with those of |hello|. For simplicity, we
// require their lengths match, which is satisfied by matching the
// DefaultCurves setting to the selection in the replacement
// ClientHello.
bb := newByteBuilder()
hello.marshalKeyShares(bb)
keyShares := bb.finish()
if len(keyShares) != len(newHello.keySharesRaw) {
return nil, errors.New("tls: ClientHello key share lengths did not match. Reconfigure DefaultCurves.")
return nil, errors.New("tls: ClientHello key share length is inconsistent with DefaultCurves setting")
}
// |newHello.keySharesRaw| aliases |ret|.
copy(newHello.keySharesRaw, keyShares)