From 48dad46a09f97f858be494b274d0942367a610d6 Mon Sep 17 00:00:00 2001 From: Tom Deseyn Date: Thu, 4 Jun 2020 16:53:27 +0200 Subject: [PATCH] SafeProcessHandle.Unix: fix missing DangerousRelease (#37412) Because the SafeProcessHandle was not owned, ReleaseHandle was not called, causing the wrapped SafeWaitHandle to never release its resources. --- .../src/Microsoft/Win32/SafeHandles/SafeProcessHandle.Unix.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/libraries/System.Diagnostics.Process/src/Microsoft/Win32/SafeHandles/SafeProcessHandle.Unix.cs b/src/libraries/System.Diagnostics.Process/src/Microsoft/Win32/SafeHandles/SafeProcessHandle.Unix.cs index 798e8f23d305c3..3eec27d88268d7 100644 --- a/src/libraries/System.Diagnostics.Process/src/Microsoft/Win32/SafeHandles/SafeProcessHandle.Unix.cs +++ b/src/libraries/System.Diagnostics.Process/src/Microsoft/Win32/SafeHandles/SafeProcessHandle.Unix.cs @@ -28,7 +28,7 @@ public sealed partial class SafeProcessHandle : SafeHandleZeroOrMinusOneIsInvali private readonly bool _releaseRef; internal SafeProcessHandle(int processId, SafeWaitHandle handle) : - this(handle.DangerousGetHandle(), ownsHandle: false) + this(handle.DangerousGetHandle(), ownsHandle: true) { ProcessId = processId; _handle = handle;