summaryrefslogtreecommitdiffstats
path: root/ipapython
diff options
context:
space:
mode:
authorMartin Kosek <mkosek@redhat.com>2011-03-01 14:17:03 +0100
committerRob Crittenden <rcritten@redhat.com>2011-03-03 11:02:02 -0500
commitf785af4efea088466f8ec2259c79640b9f94e542 (patch)
tree9749d8138cfad8e2872e96c8c6ea6d8fe7ccc332 /ipapython
parent620903ff27b0dec764a618e2962a8623dde0febe (diff)
downloadfreeipa-f785af4efea088466f8ec2259c79640b9f94e542.tar.gz
freeipa-f785af4efea088466f8ec2259c79640b9f94e542.tar.xz
freeipa-f785af4efea088466f8ec2259c79640b9f94e542.zip
Inconsistent sysrestore file handling by IPA server installer
IPA server/replica uninstallation may fail when it tries to restore a Directory server configuration file in sysrestore directory, which was already restored before. The problem is in Directory Server uninstaller which uses and modifies its own image of sysrestore directory state instead of using the common uninstaller image. https://fedorahosted.org/freeipa/ticket/1026
Diffstat (limited to 'ipapython')
-rw-r--r--ipapython/sysrestore.py1
1 files changed, 1 insertions, 0 deletions
diff --git a/ipapython/sysrestore.py b/ipapython/sysrestore.py
index 77495b200..1025449c2 100644
--- a/ipapython/sysrestore.py
+++ b/ipapython/sysrestore.py
@@ -190,6 +190,7 @@ class FileStore:
backup_path = os.path.join(self._path, filename)
if not os.path.exists(backup_path):
logging.debug(" -> Not restoring - '%s' doesn't exist", backup_path)
+ continue
shutil.move(backup_path, path)
os.chown(path, int(uid), int(gid))