Repository navigation
os.lstat() supports dir_fd but is not in os.supports_dir_fd #134993
Description
Bug report
Bug description:
I had written a script using os.fwalk() and os.lstat(), which was working.
Then I added an if os.lstat in os.supports_dir_fd check and a fallback implementation and observed the fallback path being taken:
import os, sys
from datetime import datetime
oldest = datetime.max
newest = datetime.min
oldest_file = None
newest_file = None
if os.lstat in os.supports_dir_fd:
for dir, _, files, dirfd in os.fwalk(sys.argv[1]):
for file in files:
mtime = os.lstat(file, dir_fd=dirfd).st_mtime
date = datetime.fromtimestamp(mtime)
if date < oldest:
oldest = date
oldest_file = os.path.join(dir, file)
if date > newest:
newest = date
newest_file = os.path.join(dir, file)
else:
print('not using dir_fd')
# for dir, _, files in os.walk(sys.argv[1], followlinks=False): ...
print('Newest:', newest.strftime('%Y-%m-%d %H-%M-%S'), newest_file)
print('Oldest:', oldest.strftime('%Y-%m-%d %H-%M-%S'), oldest_file)The workaround is to check for os.stat instead (and use os.stat(..., follow_symlinks=False) for consistency).
CPython versions tested on:
3.13
Operating systems tested on:
Linux
Activity
zangjiucheng commentedon Jun 4, 2025
Hi there, I tested this code and reproduce the case as you described.
I noticed the following code in
os.py; it includes all the attributes included insupports_dir_fd.cpython/Lib/os.py
Line 120 in c211130
It included
_add("HAVE_FSTATAT","stat")but not included_add("HAVE_LSTAT","lstat"). I am uncertain whether this was intentionally designed or, in the alternative, we should incorporate it.Reference:
https://linux.die.net/man/2/lstat
pythongh-134993: Add os.lstat() to os.supports_dir_fd
vstinner commentedon Jun 5, 2025
I proposed #135188 to fix the issue.
gh-134993: Add os.lstat() to os.supports_dir_fd (#135188)
pythongh-134993: Add os.lstat() to os.supports_dir_fd (pythonGH-135188)
pythongh-134993: Add os.lstat() to os.supports_dir_fd (pythonGH-135188)
vstinner commentedon Jun 6, 2025
Fixed. Thanks for your report.
In the meanwhile, don't rely on
os.supports_dir_fdbut makes the assumption thatos.lstat()supports dir_fd.[3.13] gh-134993: Add os.lstat() to os.supports_dir_fd (GH-135188) (#…
[3.14] gh-134993: Add os.lstat() to os.supports_dir_fd (GH-135188) (#…
pythongh-134993: Add os.lstat() to os.supports_dir_fd (python#135188)
pythongh-134993: Add os.lstat() to os.supports_dir_fd (python#135188)
pythongh-134993: Add os.lstat() to os.supports_dir_fd (python#135188)
[3.12] gh-157579: Fix race condition in the cleanup of tempfile.Tempo…
pythongh-134993: Add os.lstat() to os.supports_dir_fd (python#135188)