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

Reply via email to