Fix a potential CFRelease(NULL).

net/socket/ssl_client_socket_nss.cc

Index: net/socket/ssl_client_socket_nss.cc
diff --git a/net/socket/ssl_client_socket_nss.cc b/net/socket/ssl_client_socket_nss.cc
index bd7117d77c1700ebf01002b1fe4f16ac6d44722d..b16dfdf88baab52b3bf4d70130c6a781fff65589 100644
--- a/net/socket/ssl_client_socket_nss.cc
+++ b/net/socket/ssl_client_socket_nss.cc
@@ -2383,8 +2383,11 @@ SECStatus SSLClientSocketNSS::PlatformClientAuthHandler(
         }
       }
       if (os_error == noErr) {
-        int cert_count = CFArrayGetCount(chain);
-        CFRelease(chain);
+        int cert_count = 0;
+        if (chain) {
+          CFArrayGetCount(chain);

[Ryan Sleevi, 2011/10/25 00:14:16]

nico: Unchecked commit box. Now, |cert_count| is no longer updated.

I'm assuming this should be cert_count = CFArrayGetCount(chain); ?

+          CFRelease(chain);
+        }
         that->net_log_.AddEvent(NetLog::TYPE_SSL_CLIENT_CERT_PROVIDED,
             make_scoped_refptr(new NetLogIntegerParameter("cert_count",
                                                           cert_count)));