Merge pull request '[agent/claude:opus] Fix CodeRabbit findings. Verify each against current code, f...' (#4) from agent/fix-coderabbit-findings--verify-each-aga into main
This commit is contained in:
commit
3f1f9a7d60
1 changed files with 3 additions and 0 deletions
|
|
@ -309,6 +309,9 @@ func extractTarball(tarData []byte, destDir string) (string, error) {
|
||||||
return "", err
|
return "", err
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// os.OpenFile is used deliberately here instead of coreio.Local.Create/Write
|
||||||
|
// because coreio hardcodes file permissions (0644) and we need to preserve
|
||||||
|
// the tar header's mode bits — executable binaries require 0755.
|
||||||
f, err := os.OpenFile(fullPath, os.O_CREATE|os.O_WRONLY|os.O_TRUNC, os.FileMode(hdr.Mode))
|
f, err := os.OpenFile(fullPath, os.O_CREATE|os.O_WRONLY|os.O_TRUNC, os.FileMode(hdr.Mode))
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return "", coreerr.E("extractTarball", "failed to create file "+hdr.Name, err)
|
return "", coreerr.E("extractTarball", "failed to create file "+hdr.Name, err)
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue