Use the same handle for checking and duplicating in DuplicateHandleProxy

sandbox/win/src/handle_policy.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 "sandbox/win/src/handle_policy.h"
 
 #include <string>
 
 #include "base/win/scoped_handle.h"
 #include "sandbox/win/src/broker_services.h"
@@ skipping 34 matching lines @@
                                      CASE_INSENSITIVE)) {
     return false;
   }
   if (!policy->AddRule(IPC_DUPLICATEHANDLEPROXY_TAG, &duplicate_rule)) {
     return false;
   }
   return true;
 }
 
 DWORD HandlePolicy::DuplicateHandleProxyAction(EvalResult eval_result,
                                                const ClientInfo& client_info,

[cpu_(ooo_6.6-7.5), 2014/02/11 18:30:19]

remove client info.

                                                HANDLE source_handle,
                                                DWORD target_process_id,
                                                HANDLE* target_handle,
                                                DWORD desired_access,
                                                DWORD options) {
   // The only action supported is ASK_BROKER which means duplicate the handle.
   if (ASK_BROKER != eval_result) {
     return ERROR_ACCESS_DENIED;
   }
 
   base::win::ScopedHandle remote_target_process;
   if (target_process_id != ::GetCurrentProcessId()) {
     // Sandboxed children are dynamic, so we check that manually.
     if (!BrokerServicesBase::GetInstance()->IsActiveTarget(target_process_id)) {
       return ERROR_ACCESS_DENIED;
     }
 
     remote_target_process.Set(::OpenProcess(PROCESS_DUP_HANDLE, FALSE,
                                             target_process_id));
     if (!remote_target_process.IsValid())
       return ::GetLastError();
   }
 
   // If the policy didn't block us and we have no valid target, then the broker
   // (this process) is the valid target.
   HANDLE target_process = remote_target_process.IsValid() ?
                           remote_target_process.Get() : ::GetCurrentProcess();
   DWORD result = ERROR_SUCCESS;
-  if (!::DuplicateHandle(client_info.process, source_handle, target_process,
+  if (!::DuplicateHandle(::GetCurrentProcess(), source_handle, target_process,
                          target_handle, desired_access, FALSE,
                          options)) {
     return ::GetLastError();
   }
 
   return ERROR_SUCCESS;
 }
 
 }  // namespace sandbox