Skip to content

Commit 2038955

Browse files
4a6f656cFiloSottile
authored andcommitted
crypto/tls: simplify intermediate certificate handling
The certificates argument to verifyServerCertificate must contain at least one certificate. Simplify the intermediate certificate handling code accordingly. Change-Id: I8292cdfb51f418e011d6d97f47d10b4e631aa932 Reviewed-on: https://go-review.googlesource.com/c/go/+/169657 Reviewed-by: Filippo Valsorda <[email protected]>
1 parent 2d68380 commit 2038955

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

src/crypto/tls/handshake_client.go

+1-5
Original file line numberDiff line numberDiff line change
@@ -826,11 +826,7 @@ func (c *Conn) verifyServerCertificate(certificates [][]byte) error {
826826
DNSName: c.config.ServerName,
827827
Intermediates: x509.NewCertPool(),
828828
}
829-
830-
for i, cert := range certs {
831-
if i == 0 {
832-
continue
833-
}
829+
for _, cert := range certs[1:] {
834830
opts.Intermediates.AddCert(cert)
835831
}
836832
var err error

0 commit comments

Comments
 (0)