Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(877)

Unified Diff: net/cert/internal/parsed_certificate.cc

Issue 2036033002: Add CertIssuerSourceAia: authorityInfoAccess fetching for CertPathBuilder. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@cert-parsing-path-building
Patch Set: review changes Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/cert/internal/parsed_certificate.cc
diff --git a/net/cert/internal/parsed_certificate.cc b/net/cert/internal/parsed_certificate.cc
index 0f836b20eccf3471ac6bff4b08c1f47ad686ceaf..e555c443f94cd7bce662147be47c52cc8f33db70 100644
--- a/net/cert/internal/parsed_certificate.cc
+++ b/net/cert/internal/parsed_certificate.cc
@@ -126,6 +126,16 @@ scoped_refptr<ParsedCertificate> ParsedCertificate::CreateFromCertificateData(
return nullptr;
}
+ // Authority information access.
+ if (ConsumeExtension(AuthorityInfoAccessOid(),
+ &result->unparsed_extensions_,
+ &result->authority_info_access_extension_)) {
+ if (!ParseAuthorityInfoAccess(
+ result->authority_info_access_extension_.value,
+ &result->ca_issuers_uris_, &result->ocsp_uris_))
+ return nullptr;
+ }
+
// NOTE: if additional extensions are consumed here, the verification code
// must be updated to process those extensions, since the
// VerifyNoUnconsumedCriticalExtensions uses the unparsed_extensions_

Powered by Google App Engine
This is Rietveld 408576698