Use user specific NSSDatabase in CertLoader.

chromeos/network/network_connection_handler.cc

Index: chromeos/network/network_connection_handler.cc
diff --git a/chromeos/network/network_connection_handler.cc b/chromeos/network/network_connection_handler.cc
index 5450338022eb83b13781cfb63ef671e28adeac44..3ad498fdece1db01ba9736e55a78282d5d862ad1 100644
--- a/chromeos/network/network_connection_handler.cc
+++ b/chromeos/network/network_connection_handler.cc
@@ -22,6 +22,7 @@
 #include "chromeos/network/network_state_handler.h"
 #include "chromeos/network/network_ui_data.h"
 #include "chromeos/network/shill_property_util.h"
+#include "chromeos/tpm_token_loader.h"
 #include "dbus/object_path.h"
 #include "net/cert/x509_certificate.h"
 #include "third_party/cros_system_api/dbus/service_constants.h"
@@ -191,9 +192,14 @@ void NetworkConnectionHandler::OnCertificatesLoaded(
   if (queued_connect_) {
     NET_LOG_EVENT("Connecting to Queued Network",
                   queued_connect_->service_path);
-    ConnectToNetwork(queued_connect_->service_path,
-                     queued_connect_->success_callback,
-                     queued_connect_->error_callback,
+    // Make a copy of |queued_connect_| parameters, because |queued_connect_|
+    // will get reset at the beginning of |ConnectToNetwork|.
+    std::string service_path = queued_connect_->service_path;
+    base::Closure success_callback = queued_connect_->success_callback;
+    network_handler::ErrorCallback error_callback =
+        queued_connect_->error_callback;
+
+    ConnectToNetwork(service_path, success_callback, error_callback,
                      false /* check_error_state */);
   } else if (initial_load) {
     // Once certificates have loaded, connect to the "best" available network.
@@ -424,9 +430,10 @@ void NetworkConnectionHandler::VerifyConfiguredAndConnect(
         return;
       }
 
-      pkcs11_id = CertificateIsConfigured(ui_data.get());
+      pkcs11_id = CertificateIsConfigured(ui_data.get(),
+                                          cert_loader_->cert_list());

[pneubeck (no reviews), 2014/01/24 13:18:02]

CertificateIsConfigured is private, why not move "cert_loader_->cert_list()"
there?

[tbarzic, 2014/01/25 00:26:27]

Done.

       // Ensure the certificate is available and configured.
-      if (!cert_loader_->IsHardwareBacked() || pkcs11_id.empty()) {
+      if (!cert_loader_->is_hardware_backed() || pkcs11_id.empty()) {
         ErrorCallbackForPendingRequest(service_path, kErrorCertificateRequired);
         return;
       }
@@ -440,13 +447,13 @@ void NetworkConnectionHandler::VerifyConfiguredAndConnect(
 
     // The network may not be 'Connectable' because the TPM properties are not
     // set up, so configure tpm slot/pin before connecting.
-    if (cert_loader_ && cert_loader_->IsHardwareBacked()) {
+    if (cert_loader_ && cert_loader_->is_hardware_backed()) {
       // Pass NULL if pkcs11_id is empty, so that it doesn't clear any
       // previously configured client cert.
       client_cert::SetShillProperties(
           client_cert_type,
           base::IntToString(cert_loader_->tpm_token_slot_id()),
-          cert_loader_->tpm_user_pin(),
+          TPMTokenLoader::Get()->tpm_user_pin(),

[pneubeck (no reviews), 2014/01/24 13:18:02]

will this change in the future too? so that there are multiple loaded
tpmtokens/one per user ?

[tbarzic, 2014/01/25 00:26:27]

Probably not.
We need TPMTokenLoader to load chaps module and initialize singleton
NSSCertDatabase (which should be gone soon). It's not really user specific :)

           pkcs11_id.empty() ? NULL : &pkcs11_id,
           &config_properties);
     }
@@ -621,12 +628,14 @@ void NetworkConnectionHandler::CheckAllPendingRequests() {
 }
 
 std::string NetworkConnectionHandler::CertificateIsConfigured(
-    NetworkUIData* ui_data) {
+    NetworkUIData* ui_data,
+    const net::CertificateList& cert_list) {
   if (ui_data->certificate_pattern().Empty())
     return std::string();
   // Find the matching certificate.
   scoped_refptr<net::X509Certificate> matching_cert =
-      client_cert::GetCertificateMatch(ui_data->certificate_pattern());
+      client_cert::GetCertificateMatch(ui_data->certificate_pattern(),
+                                       cert_list);
   if (!matching_cert.get())
     return std::string();
   return CertLoader::GetPkcs11IdForCert(*matching_cert.get());