try to delete exec fifo file when failure in creation

Signed-off-by: lifubang <lifubang@acmcoder.com>
thaJeztah · Jun 10, 2024 · e729452 · e729452
1 parent 349e5ab
commit e729452
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/libcontainer/container_linux.go b/libcontainer/container_linux.go
@@ -406,7 +406,7 @@ func (c *Container) Signal(s os.Signal) error {
 	return nil
 }
 
-func (c *Container) createExecFifo() error {
+func (c *Container) createExecFifo() (retErr error) {
 	rootuid, err := c.Config().HostRootUID()
 	if err != nil {
 		return err
@@ -423,6 +423,11 @@ func (c *Container) createExecFifo() error {
 	if err := unix.Mkfifo(fifoName, 0o622); err != nil {
 		return &os.PathError{Op: "mkfifo", Path: fifoName, Err: err}
 	}
+	defer func() {
+		if retErr != nil {
+			os.Remove(fifoName)
+		}
+	}()
 	// Ensure permission bits (can be different because of umask).
 	if err := os.Chmod(fifoName, 0o622); err != nil {
 		return err