IntenseWebs/freeipa
3
0
Fork 0
mirror of https://salsa.debian.org/freeipa-team/freeipa.git synced 2025-02-25 18:55:28 -06:00
Code Issues Packages Projects Releases Wiki Activity

sysrestore: copy files instead of moving them to avoind SELinux issues

Browse Source 
Copying files restores SELinux context.

https://fedorahosted.org/freeipa/ticket/4923

Reviewed-By: Alexander Bokovoy <abokovoy@redhat.com>
This commit is contained in:
Martin Basti 2015-07-15 16:20:59 +02:00 committed by Petr Vobornik
parent f1f3ef478d
commit 9f70128353
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
ipapython/sysrestore.py
View File

@ -186,7 +186,9 @@ class FileStore:
if new_path is not None: if new_path is not None:
path = new_path path = new_path
shutil.move(backup_path, path) shutil.copy(backup_path, path) # SELinux needs copy
os.remove(backup_path)
os.chown(path, int(uid), int(gid)) os.chown(path, int(uid), int(gid))
os.chmod(path, int(mode)) os.chmod(path, int(mode))
@ -217,7 +219,9 @@ class FileStore:
root_logger.debug(" -> Not restoring - '%s' doesn't exist", backup_path) root_logger.debug(" -> Not restoring - '%s' doesn't exist", backup_path)
continue continue
shutil.move(backup_path, path) shutil.copy(backup_path, path) # SELinux needs copy
os.remove(backup_path)
os.chown(path, int(uid), int(gid)) os.chown(path, int(uid), int(gid))
os.chmod(path, int(mode)) os.chmod(path, int(mode))