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

Unified Diff: net/cert/cert_verify_proc_openssl.cc

Issue 137553004: NSS Cros multiprofile: trust roots added by a profile shouldn't apply to other profiles. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: update ProfileIOData comment Created 6 years, 10 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/cert_verify_proc_openssl.cc
diff --git a/net/cert/cert_verify_proc_openssl.cc b/net/cert/cert_verify_proc_openssl.cc
index 3bff531a4991d925030d1eea3e3d0a64dca80ce9..0f21daa13f3c78a39b6dd7418f5b690389829374 100644
--- a/net/cert/cert_verify_proc_openssl.cc
+++ b/net/cert/cert_verify_proc_openssl.cc
@@ -140,15 +140,9 @@ void GetCertChainInfo(X509_STORE_CTX* store_ctx,
} else {
root = verified_chain.back();
}
- const CertificateList& temporary_roots =
- TestRootCerts::GetInstance()->temporary_roots();
- for (size_t i = 0; i < temporary_roots.size(); ++i) {
- if (X509Certificate::IsSameOSCert(
- root, temporary_roots[i]->os_cert_handle())) {
+ TestRootCerts* root_certs = TestRootCerts::GetInstance();
+ if (root_certs->Contains(root))
verify_result->is_issued_by_known_root = false;
- break;
- }
- }
}
}
}

Powered by Google App Engine
This is Rietveld 408576698