Sandbox: Don't CloseHandle after calling DuplicateHandle

sandbox/win/src/process_thread_policy.cc

Index: sandbox/win/src/process_thread_policy.cc
diff --git a/sandbox/win/src/process_thread_policy.cc b/sandbox/win/src/process_thread_policy.cc
index f2029ba6819b56498b47882928e72c45adfd5d37..065359161f4f50cd9ad67f0d2054742a5962116c 100644
--- a/sandbox/win/src/process_thread_policy.cc
+++ b/sandbox/win/src/process_thread_policy.cc
@@ -124,7 +124,6 @@ NTSTATUS ProcessPolicy::OpenThreadAction(const ClientInfo& client_info,
     if (!::DuplicateHandle(::GetCurrentProcess(), local_handle,
                            client_info.process, handle, 0, FALSE,
                            DUPLICATE_CLOSE_SOURCE | DUPLICATE_SAME_ACCESS)) {
-      ::CloseHandle(local_handle);
       return STATUS_ACCESS_DENIED;
     }
   }
@@ -156,7 +155,6 @@ NTSTATUS ProcessPolicy::OpenProcessAction(const ClientInfo& client_info,
     if (!::DuplicateHandle(::GetCurrentProcess(), local_handle,
                            client_info.process, handle, 0, FALSE,
                            DUPLICATE_CLOSE_SOURCE | DUPLICATE_SAME_ACCESS)) {
-      ::CloseHandle(local_handle);
       return STATUS_ACCESS_DENIED;
     }
   }
@@ -182,7 +180,6 @@ NTSTATUS ProcessPolicy::OpenProcessTokenAction(const ClientInfo& client_info,
     if (!::DuplicateHandle(::GetCurrentProcess(), local_handle,
                            client_info.process, handle, 0, FALSE,
                            DUPLICATE_CLOSE_SOURCE | DUPLICATE_SAME_ACCESS)) {
-      ::CloseHandle(local_handle);
       return STATUS_ACCESS_DENIED;
     }
   }
@@ -208,7 +205,6 @@ NTSTATUS ProcessPolicy::OpenProcessTokenExAction(const ClientInfo& client_info,
     if (!::DuplicateHandle(::GetCurrentProcess(), local_handle,
                            client_info.process, handle, 0, FALSE,
                            DUPLICATE_CLOSE_SOURCE | DUPLICATE_SAME_ACCESS)) {
-      ::CloseHandle(local_handle);
       return STATUS_ACCESS_DENIED;
     }
   }