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

Unified Diff: net/base/cert_verifier.cc

Issue 3112013: Move chain building/verification out of X509Certificate (Closed)
Patch Set: Rebase to trunk - Without OpenSSL fixes Created 10 years, 2 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/base/cert_verifier.cc
diff --git a/net/base/cert_verifier.cc b/net/base/cert_verifier.cc
index 4e941335653adea573ae13ae3dadcb18ebed3c01..92245f0c9f0e54ac8e01d957f569a5e1a75fd39a 100644
--- a/net/base/cert_verifier.cc
+++ b/net/base/cert_verifier.cc
@@ -13,6 +13,7 @@
#include "net/base/cert_verify_result.h"
#include "net/base/net_errors.h"
#include "net/base/x509_certificate.h"
+#include "net/base/x509_chain.h"
namespace net {
@@ -37,7 +38,7 @@ class CertVerifier::Request :
void DoVerify() {
// Running on the worker thread
- error_ = cert_->Verify(hostname_, flags_, &result_);
+ error_ = x509_chain::VerifySSLServer(cert_, hostname_, flags_, &result_);
#if defined(USE_NSS)
// Detach the thread from NSPR.
// Calling NSS functions attaches the thread to NSPR, which stores
@@ -134,7 +135,7 @@ int CertVerifier::Verify(X509Certificate* cert,
// Do a synchronous verification.
if (!callback) {
CertVerifyResult result;
- int rv = cert->Verify(hostname, flags, &result);
+ int rv = x509_chain::VerifySSLServer(cert, hostname, flags, &result);
*verify_result = result;
return rv;
}
« no previous file with comments | « net/base/cert_verifier.h ('k') | net/base/x509_certificate.h » ('j') | net/base/x509_chain.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698