Skip to content

Commit

Permalink
fix: use 0600 perms for tmp files for post analyzers (#6386)
Browse files Browse the repository at this point in the history
  • Loading branch information
DmitriyLewen committed Mar 27, 2024
1 parent f148eb1 commit 9d7f5c9
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion pkg/fanal/analyzer/fs.go
Expand Up @@ -55,7 +55,8 @@ func (c *CompositeFS) CopyFileToTemp(opener Opener, info os.FileInfo) (string, e
return "", xerrors.Errorf("copy error: %w", err)
}

if err = os.Chmod(f.Name(), info.Mode()); err != nil {
// Use 0600 instead of file permissions to avoid errors when a file uses incorrect permissions (e.g. 0044).
if err = os.Chmod(f.Name(), 0600); err != nil {
return "", xerrors.Errorf("chmod error: %w", err)
}

Expand Down

0 comments on commit 9d7f5c9

Please sign in to comment.