From 6a3a28907451350d373a25d896e459ebfa7eda13 Mon Sep 17 00:00:00 2001 From: "mark@chromium.org" Date: Mon, 6 Feb 2012 22:35:05 +0000 Subject: Fixes for review comments following r120599. BUG=112443 Review URL: https://chromiumcodereview.appspot.com/9337007 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@120636 0039d316-1c4b-4281-b951-d872f2087c98 --- net/base/x509_certificate_mac.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/net/base/x509_certificate_mac.cc b/net/base/x509_certificate_mac.cc index d69b87f..d2556d7 100644 --- a/net/base/x509_certificate_mac.cc +++ b/net/base/x509_certificate_mac.cc @@ -1148,9 +1148,9 @@ int X509Certificate::VerifyInternal(const std::string& hostname, if (copy_extended_result) { CFDictionaryRef ev_dict_temp = NULL; status = copy_extended_result(trust_ref, &ev_dict_temp); - base::mac::ScopedCFTypeRef ev_dict(ev_dict_temp); + ScopedCFTypeRef ev_dict(ev_dict_temp); ev_dict_temp = NULL; - if (status == noErr) { + if (status == noErr && ev_dict) { // In 10.7.3, SecTrustCopyExtendedResult returns noErr and populates // ev_dict even for non-EV certificates, but only EV certificates // will cause ev_dict to contain kSecEVOrganizationName. In previous -- cgit v1.1