Merge pull request etcd-io#18855 from ghouscht/backport-defrag-close-…

…temp-3.4 [3.4] fix(defrag): close temp file in case of error
ivanvc · Nov 9, 2024 · c4d1505 · c4d1505
2 parents 29145e8 + d5bf838
commit c4d1505
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/mvcc/backend/backend.go b/mvcc/backend/backend.go
@@ -470,6 +470,15 @@ func (b *backend) defrag() error {
 	tdbp := temp.Name()
 	tmpdb, err := bolt.Open(tdbp, 0600, &options)
 	if err != nil {
+		temp.Close()
+		if rmErr := os.Remove(temp.Name()); rmErr != nil && b.lg != nil {
+			b.lg.Error(
+				"failed to remove temporary file",
+				zap.String("path", temp.Name()),
+				zap.Error(rmErr),
+			)
+		}
+
 		return err
 	}