From c4e4afc90eb6fd31710edb062bacfae0643d170f Mon Sep 17 00:00:00 2001 From: Michael Pratt Date: Fri, 23 Feb 2024 13:07:11 -0500 Subject: Revert "os: make FindProcess use pidfd on Linux" This reverts CL 542699. Reason for revert: Some applications assume FindProcess does not return errors. For #62654. Fixes #65866. Change-Id: Ic185a6253c8e508b08150b618c39a9905f6cdd60 Reviewed-on: https://go-review.googlesource.com/c/go/+/566476 Reviewed-by: Bryan Mills LUCI-TryBot-Result: Go LUCI Auto-Submit: Michael Pratt --- src/os/exec_unix.go | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) (limited to 'src/os/exec_unix.go') diff --git a/src/os/exec_unix.go b/src/os/exec_unix.go index 21d03da48e..2c66a8be20 100644 --- a/src/os/exec_unix.go +++ b/src/os/exec_unix.go @@ -106,14 +106,8 @@ func (p *Process) release() error { } func findProcess(pid int) (p *Process, err error) { - h, err := pidfdFind(pid) - if err == ErrProcessDone { - return nil, err - } - // Ignore all other errors from pidfdFind, - // as the callers do not expect them, and - // we can use pid anyway. - return newProcess(pid, h), nil + // NOOP for unix. + return newProcess(pid, unsetHandle), nil } func (p *ProcessState) userTime() time.Duration { -- cgit v1.3-5-g9baa