Hi, If /tmp happens to be empty os.removedirs() happily removes it...
Seen on Ubuntu Oneiric. Cheers, -- Marko Myllynen
>From 296dd30279503c2f6891cf5916a1a6e56c9512d4 Mon Sep 17 00:00:00 2001 From: Marko Myllynen <mylly...@redhat.com> Date: Thu, 22 Sep 2011 19:41:50 +0300 Subject: [PATCH] Don't remove /tmp when removing temp cert dir If /tmp happens to be empty os.removedirs() happily removes it... --- ipa-client/ipaclient/ipadiscovery.py | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/ipa-client/ipaclient/ipadiscovery.py b/ipa-client/ipaclient/ipadiscovery.py index ecd8275..9d909fd 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): -- 1.7.1
_______________________________________________ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel