Potential fix for pull request finding 'Writable file handle closed without error handling'

qmuntal · github-code-quality[bot] · web-flow · commit ba321bf7ffdc · 2026-02-10T12:29:25.000+01:00
Co-authored-by: Copilot Autofix powered by AI &lt;223894421+github-code-quality[bot]@users.noreply.github.com&gt;
diff --git a/go/cmd/bundler/main.go b/go/cmd/bundler/main.go
@@ -529,7 +529,9 @@ func extractFileFromTarball(tarballPath, destDir, targetPath, outputName string)
 			}
 
 			if _, err := io.Copy(outFile, tarReader); err != nil {
-				outFile.Close()
+				if cerr := outFile.Close(); cerr != nil {
+					return fmt.Errorf("failed to extract binary (copy error: %v, close error: %v)", err, cerr)
+				}
 				return fmt.Errorf("failed to extract binary: %w", err)
 			}
 			if err := outFile.Close(); err != nil {

Original file line number	Diff line number	Diff line change
`@@ -529,7 +529,9 @@ func extractFileFromTarball(tarballPath, destDir, targetPath, outputName string)`
`529`	`529`	`}`
`530`	`530`
`531`	`531`	`if _, err := io.Copy(outFile, tarReader); err != nil {`
`532`		`- outFile.Close()`
	`532`	`+ if cerr := outFile.Close(); cerr != nil {`
	`533`	`+ return fmt.Errorf("failed to extract binary (copy error: %v, close error: %v)", err, cerr)`
	`534`	`+ }`
`533`	`535`	`return fmt.Errorf("failed to extract binary: %w", err)`
`534`	`536`	`}`
`535`	`537`	`if err := outFile.Close(); err != nil {`