Remove unsafe registry APIs from base::win::RegKey

remoting/host/daemon_process_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 "remoting/host/daemon_process.h"
 
 #include <stdint.h>
 
 #include <memory>
 #include <utility>
@@ skipping 345 matching lines @@
 
     SECURITY_ATTRIBUTES security_attributes = {0};
     security_attributes.nLength = sizeof(security_attributes);
     security_attributes.lpSecurityDescriptor = sd.get();
     security_attributes.bInheritHandle = FALSE;
 
     HKEY key = nullptr;
     result = ::RegCreateKeyEx(
         root.Handle(), kPairingRegistrySecretsKeyName, 0, nullptr, 0,
         KEY_READ | KEY_WRITE, &security_attributes, &key, &disposition);
-    privileged.Set(key);
+    if (result == ERROR_SUCCESS) {
+      RegCloseKey(key);
+      privileged.Open(root.Handle(), kPairingRegistrySecretsKeyName,
+                      KEY_READ | KEY_WRITE);
+    }
   }
 
   if (result != ERROR_SUCCESS) {
     ::SetLastError(result);
     PLOG(ERROR) << "Failed to open or create HKLM\\" << kPairingRegistryKeyName
                 << "\\" << kPairingRegistrySecretsKeyName;
     return false;
   }
 
-  pairing_registry_privileged_key_.Set(privileged.Take());
-  pairing_registry_unprivileged_key_.Set(unprivileged.Take());
+  pairing_registry_privileged_key_ = std::move(privileged);
+  pairing_registry_unprivileged_key_ = std::move(unprivileged);
   return true;
 }
 
 }  // namespace remoting