dir() can crash with cyclic __bases__
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 77.2k
- Forks
- 35.9k
- PR-Merge-Kennzahlen
- PR-Kennzahlen ausstehend
Beschreibung
Crash report
What happened?
dir() can cause a native stack overflow when an object's __class__
provides a cyclic __bases__ attribute.
Minimal reproducer:
class Fake:
pass
a = Fake()
a.__bases__ = (a,)
class C:
@property
def __class__(self):
return a
print("entering", flush=True)
dir(C())
print("survived", flush=True)
Observed Result
On CPython 3.16.0a0 built from commit 5107fd700d7:
entering
timeout: the monitored command dumped core
Segmentation fault
rc=139
The crash occurs because object.__dir__() obtains the object's __class__ and merge_class_dict() recursively traverses __bases__ without a recursion guard. A cyclic __bases__ therefore causes unbounded native recursion and eventually a SIGSEGV.
The analogous __bases__ traversal in abstract_issubclass() already uses _Py_EnterRecursiveCall().
I also verified that adding a recursion guard to merge_class_dict() changes the failure from a native crash to a catchable exception:
RecursionError: Stack overflow (used 8120 kB) in __bases__
A regression test covering the cyclic __bases__ case was also added to Lib/test/test_builtin.py and passes with:
./python -m test test_builtin -m test_dir
== Tests result: SUCCESS ==
1 test OK.
CPython versions tested on:
CPython main branch
Operating systems tested on:
Linux
Output from running 'python -VV' on the command line:
Python 3.16.0a0 (heads/investigate-0071-dir-cyclic-bases-dirty:5107fd700d7, Aug 9 2026, 21:10:20) [GCC 13.3.0]
Linked PRs
- gh-155453
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit merge_class_dict() und dem im Issue beschriebenen Reproduzierer für zyklische bases, und lies dann den Regressionstest in Lib/test/test_builtin.py. Führe ./python -m test test_builtin -m test_dir aus; damit ist die Aufgabe erledigt, wenn der Fall einen abfangbaren RecursionError auslöst, anstatt einen nativen Stack Overflow zu verursachen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- compilers
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 25/100