diff options
| author | Martin Basti <mbasti@redhat.com> | 2015-12-09 13:40:04 +0100 |
|---|---|---|
| committer | Tomas Babej <tbabej@redhat.com> | 2015-12-14 14:57:26 +0100 |
| commit | 4272ba40ea909b1f783a6fada5b1eebb6efbdf93 (patch) | |
| tree | e02fbd38e71f71a4848a62cf761cbfe1b964d192 /ipaplatform | |
| parent | 5e2cd38ab998230aa81cd07196edfba2b62cb58d (diff) | |
| download | freeipa-4272ba40ea909b1f783a6fada5b1eebb6efbdf93.tar.gz freeipa-4272ba40ea909b1f783a6fada5b1eebb6efbdf93.tar.xz freeipa-4272ba40ea909b1f783a6fada5b1eebb6efbdf93.zip | |
Explicitly call chmod on newly created directories
Without calling os.chmod(), umask is effective and may cause that
directory is created with permission that causes failure.
This can be related to https://fedorahosted.org/freeipa/ticket/5520
Reviewed-By: Tomas Babej <tbabej@redhat.com>
Diffstat (limited to 'ipaplatform')
| -rw-r--r-- | ipaplatform/base/services.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/ipaplatform/base/services.py b/ipaplatform/base/services.py index 06224ff09..41b9654c9 100644 --- a/ipaplatform/base/services.py +++ b/ipaplatform/base/services.py @@ -420,6 +420,7 @@ class SystemdService(PlatformService): try: if not ipautil.dir_exists(srv_tgt): os.mkdir(srv_tgt) + os.mkdir(srv_tgt, 0o755) if os.path.exists(srv_lnk): # Remove old link os.unlink(srv_lnk) |