mktemp() shouldn't rely on os.path.exists(), which can return False if
the file is a symlink. Instead, use os.lstat directly, if it exists;
fall back on os.stat or the built-in open. Thanks to Iustin Pop.
diff --git a/Lib/tempfile.py b/Lib/tempfile.py
index 1405a7f..ef6a1c5 100644
--- a/Lib/tempfile.py
+++ b/Lib/tempfile.py
@@ -84,6 +84,28 @@
_once_lock = _allocate_lock()
+if hasattr(_os, "lstat"):
+ _stat = _os.lstat
+elif hasattr(_os, "stat"):
+ _stat = _os.stat
+else:
+ # Fallback. All we need is something that raises os.error if the
+ # file doesn't exist.
+ def _stat(fn):
+ try:
+ f = open(fn)
+ except IOError:
+ raise _os.error
+ f.close()
+
+def _exists(fn):
+ try:
+ _stat(fn)
+ except _os.error:
+ return False
+ else:
+ return True
+
class _RandomNameSequence:
"""An instance of _RandomNameSequence generates an endless
sequence of unpredictable strings which can safely be incorporated
@@ -339,7 +361,7 @@
for seq in xrange(TMP_MAX):
name = names.next()
file = _os.path.join(dir, prefix + name + suffix)
- if not _os.path.exists(file):
+ if not _exists(file):
return file
raise IOError, (_errno.EEXIST, "No usable temporary filename found")