Check X509Certificate::CreateFromHandle result.

net/ssl/client_cert_store_win.cc

 // Copyright 2013 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/ssl/client_cert_store_win.h"
 
 #include <algorithm>
 #include <string>
 
 #define SECURITY_WIN32  // Needs to be defined before including security.h
@@ skipping 131 matching lines @@
       intermediates.pop_back();
     }
 
     // TODO(svaldez): cert currently wraps cert_context2 which may be backed
     // by a smartcard with threading difficulties. Instead, create a fresh
     // X509Certificate with CreateFromBytes and route cert_context2 into the
     // SSLPrivateKey. Probably changing CertificateList to be a
     // pair<X509Certificate, SSLPrivateKeyCallback>.
     scoped_refptr<X509Certificate> cert = X509Certificate::CreateFromHandle(
         cert_context2, intermediates);
-    selected_certs->push_back(cert);
+    if (cert)
+      selected_certs->push_back(std::move(cert));
     CertFreeCertificateContext(cert_context2);
     for (size_t i = 0; i < intermediates.size(); ++i)
       CertFreeCertificateContext(intermediates[i]);
   }
 
   std::sort(selected_certs->begin(), selected_certs->end(),
             x509_util::ClientCertSorter());
 }
 
 }  // namespace
@@ skipping 67 matching lines @@
     // copy).
     if (!CertFreeCertificateContext(cert))
       return false;
   }
 
   GetClientCertsImpl(test_store.get(), request, selected_certs);
   return true;
 }
 
 }  // namespace net