Update pkg/distribution/packaging/dirtar.go

ilopezluna · Copilot · web-flow · commit f9cdf1ce8da8 · 2025-10-16T19:28:31.000+02:00
Co-authored-by: Copilot &lt;175728472+Copilot@users.noreply.github.com&gt;
diff --git a/pkg/distribution/packaging/dirtar.go b/pkg/distribution/packaging/dirtar.go
@@ -79,12 +79,16 @@ func CreateDirectoryTarArchive(dirPath string) (string, error) {
 			if err != nil {
 				return fmt.Errorf("open file %s: %w", path, err)
 			}
-			defer file.Close()
+
 
 			// Copy file contents
 			if _, err := io.Copy(tw, file); err != nil {
+				file.Close()
 				return fmt.Errorf("write tar content for %s: %w", path, err)
 			}
+			if err := file.Close(); err != nil {
+				return fmt.Errorf("close file %s: %w", path, err)
+			}
 		}
 
 		return nil

Original file line number	Diff line number	Diff line change
`@@ -79,12 +79,16 @@ func CreateDirectoryTarArchive(dirPath string) (string, error) {`
`79`	`79`	`if err != nil {`
`80`	`80`	`return fmt.Errorf("open file %s: %w", path, err)`
`81`	`81`	`}`
`82`		`- defer file.Close()`
	`82`	`+`
`83`	`83`
`84`	`84`	`// Copy file contents`
`85`	`85`	`if _, err := io.Copy(tw, file); err != nil {`
	`86`	`+ file.Close()`
`86`	`87`	`return fmt.Errorf("write tar content for %s: %w", path, err)`
`87`	`88`	`}`
	`89`	`+ if err := file.Close(); err != nil {`
	`90`	`+ return fmt.Errorf("close file %s: %w", path, err)`
	`91`	`+ }`
`88`	`92`	`}`
`89`	`93`
`90`	`94`	`return nil`