qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-11-26 15:06:52 -07:00
Code Releases Activity

os: don't try to signal PID -1 on Unix

Browse Source 
This restores behavior that we lost in CL 588675.

Fixes #68496

Change-Id: I1740986bed647835986d54109071b7a6b37413d5
Reviewed-on: https://go-review.googlesource.com/c/go/+/599015
Reviewed-by: Cherry Mui <cherryyz@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Auto-Submit: Ian Lance Taylor <iant@google.com>
Commit-Queue: Ian Lance Taylor <iant@google.com>
This commit is contained in:
Ian Lance Taylor 2024-07-17 09:23:09 -07:00 committed by Gopher Robot
parent 87abb4afb6
commit 420037b16d
2 changed files with 15 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/os/exec_unix.go
View File

@ -103,6 +103,9 @@ func (p *Process) signal(sig Signal) error {
} }
func (p *Process) pidSignal(s syscall.Signal) error { func (p *Process) pidSignal(s syscall.Signal) error {
if p.Pid == pidReleased {
return errors.New("os: process already released")
}
if p.Pid == pidUnset { if p.Pid == pidUnset {
return errors.New("os: process not initialized") return errors.New("os: process not initialized")
} }

11
src/os/exec_unix_test.go
View File

@ -76,3 +76,14 @@ func TestUNIXProcessAlive(t *testing.T) {
t.Errorf("OS reported error for running process: %v", err) t.Errorf("OS reported error for running process: %v", err)
} }
} }
func TestProcessBadPID(t *testing.T) {
p, err := FindProcess(-1)
if err != nil {
t.Fatalf("unexpected FindProcess error: %v", err)
}
err = p.Signal(syscall.Signal(0))
if err == nil {
t.Error("p.Signal succeeded unexpectedly")
}
}