summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJohn Estabrook <jestabro@sentrium.io>2019-11-25 14:07:43 -0600
committerJohn Estabrook <jestabro@sentrium.io>2019-11-25 14:07:43 -0600
commit3d2cdbdbaadc5a6c7ad50254f61744c222dae438 (patch)
treedbc5df8c5158ed9268bd42220db279cf9be05fe7
parentfc03e0fdf3e7389ee668d66cf7224df2aaaaa95b (diff)
downloadvyos-1x-3d2cdbdbaadc5a6c7ad50254f61744c222dae438.tar.gz
vyos-1x-3d2cdbdbaadc5a6c7ad50254f61744c222dae438.zip
T1824: restore original umask
-rw-r--r--python/vyos/migrator.py7
1 files changed, 5 insertions, 2 deletions
diff --git a/python/vyos/migrator.py b/python/vyos/migrator.py
index 281c247db..f05228041 100644
--- a/python/vyos/migrator.py
+++ b/python/vyos/migrator.py
@@ -80,15 +80,18 @@ class Migrator(object):
"""
self._log_file = os.path.join(vyos.defaults.directories['config'],
'vyos-migrate.log')
- # on creation, allow write permission for cfg_group
- os.umask(0o113)
+ # on creation, allow write permission for cfg_group;
+ # restore original umask on exit
+ mask = os.umask(0o113)
try:
log = open('{0}'.format(self._log_file), 'w')
log.write("List of executed migration scripts:\n")
except Exception as e:
+ os.umask(mask)
print("Logging error: {0}".format(e))
return None
+ os.umask(mask)
return log
def run_migration_scripts(self, config_file_versions, system_versions):