Fix small race in the sandbox

sandbox/win/src/sync_policy.cc

 // Copyright (c) 2006-2008 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 <string>
 
 #include "sandbox/win/src/sync_policy.h"
 
 #include "base/logging.h"
 #include "sandbox/win/src/ipc_tags.h"
@@ skipping 62 matching lines @@
     return false;
 
   HANDLE local_handle = ::CreateEvent(NULL, manual_reset, initial_state,
                                      event_name.c_str());
   if (NULL == local_handle)
     return ::GetLastError();
 
   if (!::DuplicateHandle(::GetCurrentProcess(), local_handle,
                          client_info.process, handle, 0, FALSE,
                          DUPLICATE_CLOSE_SOURCE | DUPLICATE_SAME_ACCESS)) {
-    ::CloseHandle(local_handle);
     return ERROR_ACCESS_DENIED;
   }
   return ERROR_SUCCESS;
 }
 
 DWORD SyncPolicy::OpenEventAction(EvalResult eval_result,
                                   const ClientInfo& client_info,
                                   const std::wstring &event_name,
                                   uint32 desired_access,
                                   uint32 inherit_handle,
                                   HANDLE *handle) {
   // The only action supported is ASK_BROKER which means create the requested
   // file as specified.
   if (ASK_BROKER != eval_result)
     return false;
 
   HANDLE local_handle = ::OpenEvent(desired_access, FALSE,
                                     event_name.c_str());
   if (NULL == local_handle)
     return ::GetLastError();
 
   if (!::DuplicateHandle(::GetCurrentProcess(), local_handle,
                          client_info.process, handle, 0, inherit_handle,
                          DUPLICATE_CLOSE_SOURCE | DUPLICATE_SAME_ACCESS)) {
-    ::CloseHandle(local_handle);
     return ERROR_ACCESS_DENIED;
   }
   return ERROR_SUCCESS;
 }
 
 }  // namespace sandbox