]> Cypherpunks repositories - gostls13.git/commit
crypto/x509/pkix: make 'v1' the default CRL version.
authorAdam Langley <agl@golang.org>
Thu, 10 Mar 2016 22:25:50 +0000 (14:25 -0800)
committerAdam Langley <agl@golang.org>
Thu, 10 Mar 2016 23:48:16 +0000 (23:48 +0000)
commit09d40378b946f363c9c351c2fdc8be30a09b238d
tree8e4e22f01b3f05693caafd461b5b983aa1c9c5d5
parent1b8d4caddb8852829ce7e53d0fe6b153051716aa
crypto/x509/pkix: make 'v1' the default CRL version.

PKIX versions are off-by-one, so v1 is actually a zero on the wire, v2
is a one, and so on.

The RFC says that the version in a CRL is optional, but doesn't say what
the default is. Since v2 is the only accepted version, I had made the
default v2. However, OpenSSL considers the default to be v1. Also, if
the default is v2 and the element is optional then we'll never actually
write v2 on the wire. That's contrary to the RFC which clearly assumes
that v2 will be expressed on the wire in some cases.

Therefore, this change aligns with OpenSSL and assumes that v1 is the
default CRL version.

Fixes #13931

[1] https://tools.ietf.org/html/rfc5280#section-5.1

Change-Id: Ic0f638ebdd21981d92a99a882affebf3a77ab71a
Reviewed-on: https://go-review.googlesource.com/20544
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
src/crypto/x509/pkix/pkix.go