Skip to content

Commit

Permalink
Merge pull request #2020 from kevpar/jc-leak
Browse files Browse the repository at this point in the history
Fix process handle leak when launching a job container
  • Loading branch information
kevpar authored Feb 8, 2024
2 parents 5921abb + 976716e commit 9aabef8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/exec/exec.go
Original file line number Diff line number Diff line change
Expand Up @@ -209,8 +209,8 @@ func (e *Exec) Start() error {
if err != nil {
return fmt.Errorf("failed to create process: %w", err)
}
// Don't need the thread handle for anything.
defer func() {
_ = windows.CloseHandle(windows.Handle(pi.Process))
_ = windows.CloseHandle(windows.Handle(pi.Thread))
}()

Expand Down

0 comments on commit 9aabef8

Please sign in to comment.