Add debugging information to the SSL blocking page

net/cert/x509_certificate_win.cc

 // Copyright (c) 2012 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/cert/x509_certificate.h"
 
 #include <blapi.h>  // Implement CalculateChainFingerprint() with NSS.
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 217 matching lines @@
 
   // Note: |store| is explicitly not released, as the call to CertCloseStore()
   // when |store| goes out of scope will not actually free the store. Instead,
   // the store will be freed when |primary_cert| is freed.
   return primary_cert;
 }
 
 // static
 bool X509Certificate::GetDEREncoded(X509Certificate::OSCertHandle cert_handle,
                                     std::string* encoded) {
-  if (!cert_handle->pbCertEncoded || !cert_handle->cbCertEncoded)
+  if (!cert_handle || !cert_handle->pbCertEncoded ||
+      !cert_handle->cbCertEncoded)

[agl, 2014/09/23 20:30:00]

this needs { } now that it's a multi-line conditional.

[felt, 2014/09/24 03:53:48]

Done.

     return false;
   encoded->assign(reinterpret_cast<char*>(cert_handle->pbCertEncoded),
                   cert_handle->cbCertEncoded);
   return true;
 }
 
 // static
 bool X509Certificate::IsSameOSCert(X509Certificate::OSCertHandle a,
                                    X509Certificate::OSCertHandle b) {
   DCHECK(a && b);
@@ skipping 196 matching lines @@
     if (IsCertNameBlobInIssuerList(&(*it)->pCertInfo->Issuer,
                                    valid_issuers)) {
       return true;
     }
   }
 
   return false;
 }
 
 }  // namespace net