commit | 513f0ea8cd1ca57ec833ef6343b44656e96cd4b7 | [log] [tgz] |
---|---|---|
author | David Benjamin <davidben@chromium.org> | Thu Apr 02 19:33:31 2015 -0400 |
committer | Adam Langley <agl@google.com> | Mon Apr 06 17:38:03 2015 +0000 |
tree | 4573ea26bddba869cc1f224645537f8d32e0a51c | |
parent | d0c6148ae557a1c6b49552cb10d9f6bc8e228f6b [diff] [blame] |
Test that bad Finished messages are rejected. That's a pretty obvious thing to test. I'm not sure how we forgot that one. Change-Id: I7e1a7df6c6abbdd587e0f7723117f50d09faa5c4 Reviewed-on: https://boringssl-review.googlesource.com/4211 Reviewed-by: Adam Langley <agl@google.com>
diff --git a/ssl/test/runner/handshake_server.go b/ssl/test/runner/handshake_server.go index 1ab4c5f..5db0487 100644 --- a/ssl/test/runner/handshake_server.go +++ b/ssl/test/runner/handshake_server.go
@@ -846,6 +846,9 @@ finished := new(finishedMsg) finished.verifyData = hs.finishedHash.serverSum(hs.masterSecret) + if c.config.Bugs.BadFinished { + finished.verifyData[0]++ + } c.serverVerify = append(c.serverVerify[:0], finished.verifyData...) hs.finishedBytes = finished.marshal() hs.writeServerHash(hs.finishedBytes)