_/aerc
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

correct permission check on accounts.conf

Browse source
This commit is contained in:
Bor Grošelj Simić 2021-01-13 03:46:56 +01:00 committed by Reto Brunner
parent d344ceecfe
commit bbe8ba5b31
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
config/config.go
View file

@ -636,9 +636,8 @@ func checkConfigPerms(filename string) error {
return nil // disregard absent files return nil // disregard absent files
} }
perms := info.Mode().Perm() perms := info.Mode().Perm()
goPerms := perms >> 3
// group or others have read access // group or others have read access
if goPerms&0x44 != 0 { if perms&044 != 0 {
fmt.Fprintf(os.Stderr, "The file %v has too open permissions.\n", filename) fmt.Fprintf(os.Stderr, "The file %v has too open permissions.\n", filename)
fmt.Fprintln(os.Stderr, "This is a security issue (it contains passwords).") fmt.Fprintln(os.Stderr, "This is a security issue (it contains passwords).")
fmt.Fprintf(os.Stderr, "To fix it, run `chmod 600 %v`\n", filename) fmt.Fprintf(os.Stderr, "To fix it, run `chmod 600 %v`\n", filename)