Skip to content

Commit 65042de

Browse files
authored
Merge pull request #154 from 3u13r/euler/always-verify-crl
GetCrlAndCheckRoot: always verify CRL
2 parents e5df259 + 5712b21 commit 65042de

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

verify/verify.go

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ func validateAmdLocation(name pkix.Name, role string) error {
118118
if err := checkSingletonList(name.Organization, "organization", "organizations", "Advanced Micro Devices"); err != nil {
119119
return err
120120
}
121-
return checkSingletonList(name.OrganizationalUnit, "organizational unit", "organizational uints", "Engineering")
121+
return checkSingletonList(name.OrganizationalUnit, "organizational unit", "organizational units", "Engineering")
122122
}
123123

124124
func validateRootX509(productLine string, x *x509.Certificate, version int, role, cn string) error {
@@ -301,6 +301,9 @@ func GetCrlAndCheckRoot(r *trust.AMDRootCerts, opts *Options) (*x509.RevocationL
301301
getter = trust.DefaultHTTPSGetter()
302302
}
303303
if r.CRL != nil && opts.Now.Before(r.CRL.NextUpdate) {
304+
if err := verifyCRL(r); err != nil {
305+
return nil, err
306+
}
304307
return r.CRL, nil
305308
}
306309
var errs error

0 commit comments

Comments
 (0)