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>
diff --git a/ssl/test/runner/handshake_client.go b/ssl/test/runner/handshake_client.go index d25bbab..ab1f4dd 100644 --- a/ssl/test/runner/handshake_client.go +++ b/ssl/test/runner/handshake_client.go
@@ -59,12 +59,15 @@ 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)