Chromium Code Reviews| Index: net/base/cert_verify_proc_nss.cc |
| =================================================================== |
| --- net/base/cert_verify_proc_nss.cc (revision 152152) |
| +++ net/base/cert_verify_proc_nss.cc (working copy) |
| @@ -4,6 +4,9 @@ |
| #include "net/base/cert_verify_proc_nss.h" |
| +#include <string> |
| +#include <vector> |
| + |
| #include <cert.h> |
| #include <nss.h> |
| #include <prerror.h> |
| @@ -566,24 +569,35 @@ |
| return SECOID_AddEntry(&od); |
| } |
| -SHA1Fingerprint CertPublicKeyHash(CERTCertificate* cert) { |
| - SHA1Fingerprint hash; |
| - SECStatus rv = HASH_HashBuf(HASH_AlgSHA1, hash.data, |
| +HashValue CertPublicKeyHashSHA1(CERTCertificate* cert) { |
| + HashValue hash(HASH_VALUE_SHA1); |
| + SECStatus rv = HASH_HashBuf(HASH_AlgSHA1, hash.data(), |
| cert->derPublicKey.data, cert->derPublicKey.len); |
| DCHECK_EQ(rv, SECSuccess); |
|
Ryan Sleevi
2012/08/23 22:48:20
nit: DCHECK_EQ(expected, actual)
palmer
2012/08/29 00:01:24
Done.
|
| return hash; |
| } |
| +HashValue CertPublicKeyHashSHA256(CERTCertificate* cert) { |
| + HashValue hash(HASH_VALUE_SHA256); |
| + SECStatus rv = HASH_HashBuf(HASH_AlgSHA256, hash.data(), |
| + cert->derPublicKey.data, cert->derPublicKey.len); |
| + DCHECK_EQ(rv, SECSuccess); |
| + return hash; |
| +} |
| + |
| void AppendPublicKeyHashes(CERTCertList* cert_list, |
| CERTCertificate* root_cert, |
| - std::vector<SHA1Fingerprint>* hashes) { |
| + HashValueVector* hashes) { |
| for (CERTCertListNode* node = CERT_LIST_HEAD(cert_list); |
| !CERT_LIST_END(node, cert_list); |
| node = CERT_LIST_NEXT(node)) { |
| - hashes->push_back(CertPublicKeyHash(node->cert)); |
| + hashes->push_back(CertPublicKeyHashSHA1(node->cert)); |
| + hashes->push_back(CertPublicKeyHashSHA256(node->cert)); |
| } |
| - if (root_cert) |
| - hashes->push_back(CertPublicKeyHash(root_cert)); |
| + if (root_cert) { |
| + hashes->push_back(CertPublicKeyHashSHA1(root_cert)); |
| + hashes->push_back(CertPublicKeyHashSHA256(root_cert)); |
| + } |
| } |
| // Returns true if |cert_handle| contains a policy OID that is an EV policy |
| @@ -671,7 +685,7 @@ |
| return false; |
| } |
| - SHA1Fingerprint fingerprint = |
| + SHA1HashValue fingerprint = |
| X509Certificate::CalculateFingerprint(root_ca); |
| return metadata->HasEVPolicyOID(fingerprint, ev_policy_oid); |
| } |