Skip to content

Commit 4ba9eca

Browse files
danmoseleytmds
andauthored
SafeProcessHandle.Unix: fix missing DangerousRelease (#37412) (#37446)
Because the SafeProcessHandle was not owned, ReleaseHandle was not called, causing the wrapped SafeWaitHandle to never release its resources. Co-authored-by: Tom Deseyn <[email protected]>
1 parent 74c06a9 commit 4ba9eca

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/libraries/System.Diagnostics.Process/src/Microsoft/Win32/SafeHandles/SafeProcessHandle.Unix.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ public sealed partial class SafeProcessHandle : SafeHandleZeroOrMinusOneIsInvali
2828
private readonly bool _releaseRef;
2929

3030
internal SafeProcessHandle(int processId, SafeWaitHandle handle) :
31-
this(handle.DangerousGetHandle(), ownsHandle: false)
31+
this(handle.DangerousGetHandle(), ownsHandle: true)
3232
{
3333
ProcessId = processId;
3434
_handle = handle;

0 commit comments

Comments
 (0)