Implement LoadTemporaryRoot for Windows

net/base/x509_certificate_win.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/base/x509_certificate.h"
 
+#include "base/crypto/scoped_capi_types.h"
 #include "base/logging.h"
 #include "base/pickle.h"
 #include "base/string_tokenizer.h"
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "net/base/cert_status_flags.h"
 #include "net/base/cert_verify_result.h"
 #include "net/base/ev_root_ca_metadata.h"
 #include "net/base/net_errors.h"
 #include "net/base/scoped_cert_chain_context.h"
+#include "net/base/test_root_certs.h"
 
 #pragma comment(lib, "crypt32.lib")
 
 using base::Time;
 
 namespace net {
 
 namespace {
 
+typedef base::ScopedCAPIHandle<
+    HCERTSTORE,
+    base::CAPIDestroyerWithFlags<HCERTSTORE,
+                                 CertCloseStore, 0> > ScopedHCERTSTORE;
+
 //-----------------------------------------------------------------------------
 
 // TODO(wtc): This is a copy of the MapSecurityError function in
 // ssl_client_socket_win.cc.  Another function that maps Windows error codes
 // to our network error codes is WinInetUtil::OSErrorToNetError.  We should
 // eliminate the code duplication.
 int MapSecurityError(SECURITY_STATUS err) {
   // There are numerous security error codes, but these are the ones we thus
   // far find interesting.
   switch (err) {
@@ skipping 562 matching lines @@
           chain_para.RequestedIssuancePolicy.dwType = USAGE_MATCH_TYPE_AND;
           chain_para.RequestedIssuancePolicy.Usage.cUsageIdentifier = 1;
           chain_para.RequestedIssuancePolicy.Usage.rgpszUsageIdentifier =
               &ev_policy_oid;
           break;
         }
       }
     }
   }
 
+  // Temporary roots may not exist in any system store, and therefore may not
+  // be available for certificate chain building. Create a certificate store
+  // collection containing both the certificate chain (the cert_handle's
+  // HCERTSTORE) and the temporary root's HCERTSTORE, so that Windows will
+  // search both.

[bulach, 2010/11/17 17:17:30]

similarly here.
my (ignorable) suggestion both here and mac is to try to separate test code from
production code. right now, it's not really straightforward.
maybe even split this into:
HCERTSTORE CreateStoreForTestsIfNeeded(X509Certificate::OSCertHandle handle) {
...
}

so then it become:

ScopedHCERTSTORE scoped_test_store(CreateStoreForTestsIfNeeded(cert_handle_));

HCERTSTORE intermediates_store = scoped_test_store.get() ?
  scoped_test_store.get() : cert_handle_->hCertStore;

+  TestRootCerts* temporary_roots = TestRootCerts::GetInstance();
+  HCERTSTORE collection_store = temporary_roots->IsEmpty() ?
+      NULL : CertOpenStore(CERT_STORE_PROV_COLLECTION, 0, NULL, 0, NULL);
+  if (collection_store) {
+    // Add the temporary roots with priority 0, so that certificates from the
+    // temporary roots will be of a higher priority for chain building. If
+    // they were a lower priority, it's possible a different root certificate
+    // than the one in the temporary roots will be used, which would cause
+    // the detection logic in FixTestRootTrust() to fail.
+    BOOL ok = CertAddStoreToCollection(collection_store,
+                                       temporary_roots->temporary_roots(),
+                                       0, 0);
+    if (ok)
+      ok = CertAddStoreToCollection(collection_store,
+                                    cert_handle_->hCertStore, 0, 1);
+    if (!ok) {
+      PLOG(ERROR) << "Unable to create temporary linked certificate store";
+      CertCloseStore(collection_store, 0);
+      collection_store = NULL;
+    }
+  }
+  ScopedHCERTSTORE scoped_collection_store(collection_store);
+  HCERTSTORE intermediates_store = collection_store ?
+      collection_store : cert_handle_->hCertStore;
+
   PCCERT_CHAIN_CONTEXT chain_context;
   // IE passes a non-NULL pTime argument that specifies the current system
   // time.  IE passes CERT_CHAIN_REVOCATION_CHECK_CHAIN_EXCLUDE_ROOT as the
   // chain_flags argument.
   if (!CertGetCertificateChain(
            NULL,  // default chain engine, HCCE_CURRENT_USER
            cert_handle_,
            NULL,  // current system time
-           cert_handle_->hCertStore,  // search this store
+           intermediates_store,
            &chain_para,
            chain_flags,
            NULL,  // reserved
            &chain_context)) {
     return MapSecurityError(GetLastError());
   }
   if (chain_context->TrustStatus.dwErrorStatus &
       CERT_TRUST_IS_NOT_VALID_FOR_USAGE) {
     ev_policy_oid = NULL;
     chain_para.RequestedIssuancePolicy.Usage.cUsageIdentifier = 0;
     chain_para.RequestedIssuancePolicy.Usage.rgpszUsageIdentifier = NULL;
     CertFreeCertificateChain(chain_context);
     if (!CertGetCertificateChain(
              NULL,  // default chain engine, HCCE_CURRENT_USER
              cert_handle_,
              NULL,  // current system time
-             cert_handle_->hCertStore,  // search this store
+             intermediates_store,
              &chain_para,
              chain_flags,
              NULL,  // reserved
              &chain_context)) {
       return MapSecurityError(GetLastError());
     }
   }
   ScopedCertChainContext scoped_chain_context(chain_context);
 
+  TestRootCerts* root_certs = TestRootCerts::GetInstance();
+  root_certs->UpdateChainContext(
+      const_cast<PCERT_CHAIN_CONTEXT>(chain_context));
+
   GetCertChainInfo(chain_context, verify_result);
-
   verify_result->cert_status |= MapCertChainErrorStatusToCertStatus(
       chain_context->TrustStatus.dwErrorStatus);
 
   // Treat certificates signed using broken signature algorithms as invalid.
   if (verify_result->has_md4)
     verify_result->cert_status |= CERT_STATUS_INVALID;
 
   // Flag certificates signed using weak signature algorithms.
   if (verify_result->has_md2)
     verify_result->cert_status |= CERT_STATUS_WEAK_SIGNATURE_ALGORITHM;
@@ skipping 201 matching lines @@
   DWORD sha1_size = sizeof(sha1.data);
   rv = CryptHashCertificate(NULL, CALG_SHA1, 0, cert->pbCertEncoded,
                             cert->cbCertEncoded, sha1.data, &sha1_size);
   DCHECK(rv && sha1_size == sizeof(sha1.data));
   if (!rv)
     memset(sha1.data, 0, sizeof(sha1.data));
   return sha1;
 }
 
 }  // namespace net