Improvement to the previous fix suggested by Thomas Bellman: if the
unlink() or fdopen() fail, close the file descriptor and re-raise the
exception.
diff --git a/Lib/tempfile.py b/Lib/tempfile.py
index 140eebc..1f30126 100644
--- a/Lib/tempfile.py
+++ b/Lib/tempfile.py
@@ -129,8 +129,12 @@
if os.name == 'posix':
# Unix -- be very careful
fd = os.open(name, os.O_RDWR|os.O_CREAT|os.O_EXCL, 0700)
- os.unlink(name)
- return os.fdopen(fd, mode, bufsize)
+ try:
+ os.unlink(name)
+ return os.fdopen(fd, mode, bufsize)
+ except:
+ os.close(fd)
+ raise
else:
# Non-unix -- can't unlink file that's still open, use wrapper
file = open(name, mode, bufsize)