summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarko Myllynen <myllynen@redhat.com>2011-09-22 19:41:50 +0300
committerRob Crittenden <rcritten@redhat.com>2011-09-22 22:03:12 -0400
commit5915e478ec81c7a2baa49700de976aa949e687e1 (patch)
treedc47c5dfed7964e4082b8cbfafd32dbf8075de39
parent384d4e8f5018aa1f829139572fa29abef375c5a3 (diff)
downloadfreeipa-5915e478ec81c7a2baa49700de976aa949e687e1.tar.gz
freeipa-5915e478ec81c7a2baa49700de976aa949e687e1.tar.xz
freeipa-5915e478ec81c7a2baa49700de976aa949e687e1.zip
Don't remove /tmp when removing temp cert dir
If /tmp happens to be empty os.removedirs() happily removes it... https://fedorahosted.org/freeipa/ticket/1843
-rw-r--r--ipa-client/ipaclient/ipadiscovery.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/ipa-client/ipaclient/ipadiscovery.py b/ipa-client/ipaclient/ipadiscovery.py
index ecd8275c..9d909fd1 100644
--- a/ipa-client/ipaclient/ipadiscovery.py
+++ b/ipa-client/ipaclient/ipadiscovery.py
@@ -280,7 +280,7 @@ class IPADiscovery:
finally:
os.remove("%s/ca.crt" % temp_ca_dir)
- os.removedirs(temp_ca_dir)
+ os.rmdir(temp_ca_dir)
def ipadnssearchldap(self, tdomain):