libct: replace unix.Kill with os.Process.Signal

lifubang · kolyshkin · commit 346c80d714c3 · 2025-03-07T14:07:47.000-08:00
Because we should switch to unix.PidFDSendSignal in new kernels, it has
been supported in go runtime. We don't need to add fall back to
unix.Kill code here.

Signed-off-by: lifubang &lt;lifubang@acmcoder.com&gt;
Signed-off-by: Kir Kolyshkin &lt;kolyshkin@gmail.com&gt;
diff --git a/libcontainer/process_linux.go b/libcontainer/process_linux.go
@@ -116,11 +116,7 @@ func (p *containerProcess) startTime() (uint64, error) {
 }
 
 func (p *containerProcess) signal(sig os.Signal) error {
-	s, ok := sig.(unix.Signal)
-	if !ok {
-		return errors.New("os: unsupported signal type")
-	}
-	return unix.Kill(p.pid(), s)
+	return p.cmd.Process.Signal(sig)
 }
 
 func (p *containerProcess) externalDescriptors() []string {

Original file line number	Diff line number	Diff line change
`@@ -116,11 +116,7 @@ func (p *containerProcess) startTime() (uint64, error) {`
`116`	`116`	`}`
`117`	`117`
`118`	`118`	`func (p *containerProcess) signal(sig os.Signal) error {`
`119`		`- s, ok := sig.(unix.Signal)`
`120`		`- if !ok {`
`121`		`- return errors.New("os: unsupported signal type")`
`122`		`- }`
`123`		`- return unix.Kill(p.pid(), s)`
	`119`	`+ return p.cmd.Process.Signal(sig)`
`124`	`120`	`}`
`125`	`121`
`126`	`122`	`func (p *containerProcess) externalDescriptors() []string {`