ctr: update WritePidFile to use atomicfile

samuelkarp · thaJeztah · commit 7353c0286950 · 2023-07-14T13:25:36.000+02:00
Signed-off-by: Samuel Karp <samuelkarp@google.com> (cherry picked from commit ab53652) Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
diff --git a/cmd/ctr/commands/commands.go b/cmd/ctr/commands/commands.go
@@ -24,6 +24,8 @@ import (
 	"strings"
 
 	"github.com/containerd/containerd/defaults"
+	"github.com/containerd/containerd/pkg/atomicfile"
+
 	"github.com/urfave/cli"
 )
 
@@ -268,15 +270,14 @@ func WritePidFile(path string, pid int) error {
 	if err != nil {
 		return err
 	}
-	tempPath := filepath.Join(filepath.Dir(path), fmt.Sprintf(".%s", filepath.Base(path)))
-	f, err := os.OpenFile(tempPath, os.O_RDWR|os.O_CREATE|os.O_EXCL|os.O_SYNC, 0666)
+	f, err := atomicfile.New(path, 0o666)
 	if err != nil {
 		return err
 	}
 	_, err = fmt.Fprintf(f, "%d", pid)
-	f.Close()
 	if err != nil {
+		f.Cancel()
 		return err
 	}
-	return os.Rename(tempPath, path)
+	return f.Close()
 }

Original file line number	Diff line number	Diff line change
`@@ -24,6 +24,8 @@ import (`
`24`	`24`	`"strings"`
`25`	`25`
`26`	`26`	`"github.com/containerd/containerd/defaults"`
	`27`	`+ "github.com/containerd/containerd/pkg/atomicfile"`
	`28`	`+`
`27`	`29`	`"github.com/urfave/cli"`
`28`	`30`	`)`
`29`	`31`
`@@ -268,15 +270,14 @@ func WritePidFile(path string, pid int) error {`
`268`	`270`	`if err != nil {`
`269`	`271`	`return err`
`270`	`272`	`}`
`271`		`- tempPath := filepath.Join(filepath.Dir(path), fmt.Sprintf(".%s", filepath.Base(path)))`
`272`		`- f, err := os.OpenFile(tempPath, os.O_RDWR\|os.O_CREATE\|os.O_EXCL\|os.O_SYNC, 0666)`
	`273`	`+ f, err := atomicfile.New(path, 0o666)`
`273`	`274`	`if err != nil {`
`274`	`275`	`return err`
`275`	`276`	`}`
`276`	`277`	`_, err = fmt.Fprintf(f, "%d", pid)`
`277`		`- f.Close()`
`278`	`278`	`if err != nil {`
	`279`	`+ f.Cancel()`
`279`	`280`	`return err`
`280`	`281`	`}`
`281`		`- return os.Rename(tempPath, path)`
	`282`	`+ return f.Close()`
`282`	`283`	`}`