From 003ebf0b9dbcbfcda88f39f227906eeaf921498c Mon Sep 17 00:00:00 2001 From: Yann Leboulanger Date: Fri, 21 Aug 2015 22:04:14 +0200 Subject: [PATCH] create file directly with correct rights instead of using chmod later --- src/common/optparser.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/common/optparser.py b/src/common/optparser.py index 19b05451d..6423e6756 100644 --- a/src/common/optparser.py +++ b/src/common/optparser.py @@ -104,7 +104,8 @@ class OptionsParser: (base_dir, filename) = os.path.split(self.__filename) self.__tempfile = os.path.join(base_dir, '.' + filename) try: - f = open(self.__tempfile, 'w') + f = os.fdopen(os.open(self.__tempfile, + os.O_CREAT|os.O_WRONLY|os.O_TRUNC, 0o600), 'w') except IOError as e: return str(e) try: @@ -125,7 +126,6 @@ class OptionsParser: os.rename(self.__tempfile, self.__filename) except IOError as e: return str(e) - os.chmod(self.__filename, 0o600) def update_config(self, old_version, new_version): old_version_list = old_version.split('.') # convert '0.x.y' to (0, x, y)