This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: NamedTemporaryFile can hang on windows
Type: crash Stage: resolved
Components: Versions: Python 3.7, Python 3.6
process
Status: closed Resolution: duplicate
Dependencies: Superseder: tempfile module misinterprets access denied error on Windows
View: 22107
Assigned To: Nosy List: earonesty, martin.panter
Priority: normal Keywords:

Created on 2019-07-01 20:35 by earonesty, last changed 2022-04-11 14:59 by admin. This issue is now closed.

Messages (3)
msg347073 - (view) Author: Erik Aronesty (earonesty) * Date: 2019-07-01 20:35
Depending on the user's permissions, this code can hang, instead of raising an exception:

    from tempfile import NamedTemporaryFile
    NamedTemporaryFile(dir="/")

The problamatic code is in tempfile.py:

When encountering a "[Errno 13] Permission denied: '/tmpmcupmo_g'", the current code uses _os.access(dir, _os.W_OK) in two places to check if access is allowed to write to the directory.

On windows, _os.access does not check if the user has permission to write to the folder, it only checks if the folder is read-only (and it doesn't even do a good job at that).

So the temp file creator loops for a rather long time, and consumes a massive amount of system resources, because os.TMP_MAX on modern windows versions is 2147483647.

This article explains how to check if a directory can-write without trying to write to it:

http://blog.aaronballman.com/2011/08/how-to-check-access-rights/

Alternatively, a more careful check of the winerror return value from the open call *might* be sufficient.
msg347074 - (view) Author: Martin Panter (martin.panter) * (Python committer) Date: 2019-07-01 21:07
Perhaps a duplicate of Issue 22107?
msg347075 - (view) Author: Erik Aronesty (earonesty) * Date: 2019-07-01 21:09
yes, duplicate of https://bugs.python.org/issue22107 ... tried looking first, sry.
History
Date User Action Args
2022-04-11 14:59:17adminsetgithub: 81658
2019-07-01 21:30:09martin.pantersetsuperseder: tempfile module misinterprets access denied error on Windows
resolution: duplicate
2019-07-01 21:09:52earonestysetstatus: open -> closed

messages: + msg347075
stage: resolved
2019-07-01 21:07:32martin.pantersetnosy: + martin.panter
messages: + msg347074
2019-07-01 20:35:29earonestysettype: crash
2019-07-01 20:35:15earonestycreate