fix: cio.Cancel() should close the pipes

PR fixes #8326. Signed-off-by: Hsing-Yu (David) Chen <davidhsingyuchen@gmail.com>
containerd · Mar 31, 2023 · 82ec62b · 82ec62b
1 parent 8f756bc
commit 82ec62b
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/cio/io_unix.go b/cio/io_unix.go
@@ -98,7 +98,14 @@ func copyIO(fifos *FIFOSet, ioset *Streams) (*cio, error) {
 		config:  fifos.Config,
 		wg:      wg,
 		closers: append(pipes.closers(), fifos),
-		cancel:  cancel,
+		cancel: func() {
+			cancel()
+			for _, c := range pipes.closers() {
+				if c != nil {
+					c.Close()
+				}
+			}
+		},
 	}, nil
 }