Avoid leaking temp file if an exception is raised
authorJean Raby <jraby@inverse.ca>
Thu, 11 Apr 2013 01:33:34 +0000 (21:33 -0400)
committerAndrew Bartlett <abartlet@samba.org>
Thu, 11 Apr 2013 04:06:03 +0000 (06:06 +0200)
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Thu Apr 11 06:06:03 CEST 2013 on sn-devel-104

source4/scripting/bin/samba_dnsupdate

index 33c16ecd00f4f9ff0a4ea819ff728ea26a13a611..68b0f72151f0b89092703af1473d21cec6c06899 100755 (executable)
@@ -116,7 +116,11 @@ def get_credentials(lp):
     creds.set_machine_account(lp)
     creds.set_krb_forwardable(credentials.NO_KRB_FORWARDABLE)
     (tmp_fd, ccachename) = tempfile.mkstemp()
-    creds.get_named_ccache(lp, ccachename)
+    try:
+        creds.get_named_ccache(lp, ccachename)
+    except RuntimeError as e:
+        os.unlink(ccachename)
+        raise e
 
 
 class dnsobj(object):