crypto/x509: remove unnecessary !Empty() check

This check is already done by PeekASN1Tag.

Change-Id: Ieba0e35548f7f99bce689d29adaea6b8e471cc70
GitHub-Last-Rev: b4ef3dcc2307839cb7575cf29c3e6445b6a7520e
GitHub-Pull-Request: golang/go#59197
Reviewed-on: https://go-review.googlesource.com/c/go/+/478835
Reviewed-by: Roland Shoemaker <roland@golang.org>
Run-TryBot: Ian Lance Taylor <iant@google.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Reviewed-by: Filippo Valsorda <filippo@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Ian Lance Taylor <iant@google.com>

diff --git a/src/crypto/x509/parser.go b/src/crypto/x509/parser.go
index 5a1819841df47f7ad68ca948c864c45f4ac10c01..6695212a0bb15db3b646c67a151a124b7e193df5 100644 (file)
--- a/src/crypto/x509/parser.go
+++ b/src/crypto/x509/parser.go
@@ -342,7 +342,7 @@ func parseBasicConstraintsExtension(der cryptobyte.String) (bool, int, error) {
                }
        }
        maxPathLen := -1
-       if !der.Empty() && der.PeekASN1Tag(cryptobyte_asn1.INTEGER) {
+       if der.PeekASN1Tag(cryptobyte_asn1.INTEGER) {
                if !der.ReadASN1Integer(&maxPathLen) {
                        return false, 0, errors.New("x509: invalid basic constraints")
                }