Skip to content

Commit d13d6b3

Browse files
dmitshurbradfitz
authored andcommitted
crypto/x509: improve internal comment
Apply code review suggestion from CL 46715. The block is doing more than just checking len(r.certs) == len(tc.cns). It also verifies that certificate common names match. Change-Id: I28d6926a5da48bd8f6c80aa5e5a1ed6d4990f845 Reviewed-on: https://go-review.googlesource.com/47132 Reviewed-by: Brad Fitzpatrick <[email protected]>
1 parent dc86c9a commit d13d6b3

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/crypto/x509/root_unix_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ func TestEnvVars(t *testing.T) {
110110
t.Fatal("nil roots")
111111
}
112112

113-
// Verify len(r.certs) == len(tc.cns), otherwise report where the mismatch is.
113+
// Verify that the returned certs match, otherwise report where the mismatch is.
114114
for i, cn := range tc.cns {
115115
if i >= len(r.certs) {
116116
t.Errorf("missing cert %v @ %v", cn, i)

0 commit comments

Comments
 (0)