diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2017-07-03 14:00:47 +0200 |
|---|---|---|
| committer | Dmitry Vyukov <dvyukov@google.com> | 2017-07-03 14:00:47 +0200 |
| commit | a7b199253f7a517fd62f36a5d632efee3bac737e (patch) | |
| tree | 9f981ff2814ed7389e973c02376f8b2618224be2 /pkg/config | |
| parent | 1438a6de8130467a2fcde187ba80b8a616f8daa0 (diff) | |
all: use consistent file permissions
Currently we have unix permissions for new files/dirs
hardcoded throughout the code base. Some places use 0644,
some - 0640, some - 0600 and a variety of other constants.
Introduce osutil.MkdirAll/WriteFile that use the default
permissions and use them throughout the code base.
This makes permissions consistent and also allows to easily
change the permissions later if we change our minds.
Also merge pkg/fileutil into pkg/osutil as they become
dependent on each other. The line between them was poorly
defined anyway as both operate on files.
Diffstat (limited to 'pkg/config')
| -rw-r--r-- | pkg/config/config.go | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/pkg/config/config.go b/pkg/config/config.go index 537b9f3d6..c1a761e18 100644 --- a/pkg/config/config.go +++ b/pkg/config/config.go @@ -9,6 +9,8 @@ import ( "io/ioutil" "reflect" "strings" + + "github.com/google/syzkaller/pkg/osutil" ) func LoadFile(filename string, cfg interface{}) error { @@ -37,7 +39,7 @@ func SaveFile(filename string, cfg interface{}) error { if err != nil { return err } - return ioutil.WriteFile(filename, data, 0600) + return osutil.WriteFile(filename, data) } func checkUnknownFields(data []byte, typ reflect.Type) error { |