Fix crash on login.

components/ownership/owner_settings_service.cc

 // Copyright 2014 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 "components/ownership/owner_settings_service.h"
 
 #include <cryptohi.h>
 #include <keyhi.h>
 #include <stdint.h>
 
@@ skipping 14 matching lines @@
 namespace ownership {
 
 namespace {
 
 using ScopedSGNContext = std::unique_ptr<
     SGNContext,
     crypto::NSSDestroyer1<SGNContext, SGN_DestroyContext, PR_TRUE>>;
 
 std::unique_ptr<em::PolicyFetchResponse> AssembleAndSignPolicy(
     std::unique_ptr<em::PolicyData> policy,
-    SECKEYPrivateKey* private_key) {
+    scoped_refptr<ownership::PrivateKey> private_key) {
+  DCHECK(private_key->key());
+
   // Assemble the policy.
   std::unique_ptr<em::PolicyFetchResponse> policy_response(
       new em::PolicyFetchResponse());
   if (!policy->SerializeToString(policy_response->mutable_policy_data())) {
     LOG(ERROR) << "Failed to encode policy payload.";
     return nullptr;
   }
 
-  ScopedSGNContext sign_context(
-      SGN_NewContext(SEC_OID_PKCS1_SHA1_WITH_RSA_ENCRYPTION, private_key));
+  ScopedSGNContext sign_context(SGN_NewContext(
+      SEC_OID_PKCS1_SHA1_WITH_RSA_ENCRYPTION, private_key->key()));
   if (!sign_context) {
     NOTREACHED();
     return nullptr;
   }
 
   SECItem signature_item;
   if (SGN_Begin(sign_context.get()) != SECSuccess ||
       SGN_Update(sign_context.get(),
                  reinterpret_cast<const uint8_t*>(
                      policy_response->policy_data().c_str()),
@@ skipping 46 matching lines @@
 }
 
 bool OwnerSettingsService::AssembleAndSignPolicyAsync(
     base::TaskRunner* task_runner,
     std::unique_ptr<em::PolicyData> policy,
     const AssembleAndSignPolicyAsyncCallback& callback) {
   DCHECK(thread_checker_.CalledOnValidThread());
   if (!task_runner || !IsOwner())
     return false;
   return base::PostTaskAndReplyWithResult(
-      task_runner,
-      FROM_HERE,
-      base::Bind(
-          &AssembleAndSignPolicy, base::Passed(&policy), private_key_->key()),
+      task_runner, FROM_HERE,
+      base::Bind(&AssembleAndSignPolicy, base::Passed(&policy), private_key_),
       callback);
 }
 
 bool OwnerSettingsService::SetBoolean(const std::string& setting, bool value) {
   DCHECK(thread_checker_.CalledOnValidThread());
   base::Value in_value(value);
   return Set(setting, in_value);
 }
 
 bool OwnerSettingsService::SetInteger(const std::string& setting, int value) {
@@ skipping 34 matching lines @@
   for (std::vector<IsOwnerCallback>::iterator it(is_owner_callbacks.begin());
        it != is_owner_callbacks.end();
        ++it) {
     it->Run(is_owner);
   }
 
   OnPostKeypairLoadedActions();
 }
 
 }  // namespace ownership