realpath has a race condition which might cause it to throw spurious exceptions

Offen
#97,566 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
42/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Veraltet
Tech-Stack
python

Rechercherichtung

Beginne mit der im Issue verlinkten realpath-Reproduktion und verfolge die dort beschriebene islink/readlink-Sequenz. Bestätige den Fehler, wenn ein symlink zwischen diesen Operationen gelöscht wird; abgeschlossen ist die Aufgabe, wenn realpath in diesem Race keine unbeabsichtigte Exception mehr auslöst und eine Regressionstestabdeckung für dieses Verhalten vorhanden ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

3.10 3.11 3.12 extension-modules type-bug

Bug report

realpath tests if a file is a symlink using islink, which just returns False if the file does not exist. But then uses readlink a bit later to test the source of the link, which throws an exception if the file does not exists. If the file is a symlink and is deleted between islink and readlink, we get a spurious exception.

This reproduces the error using code copy-pasted from Python 3.9: https://gist.github.com/bitonic/e0977aaa674523a9ffe0e77a96cd6cc7 .

Your environment

  • CPython 3.9
  • ArchLinux x86_64
Vorherrschende Sprache
Python
Sterne
77.2k
Forks
36k
Ø Merge
1 T. 9 Std.
Gemergte PRs (30 T.)
558

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus python/cpython

Alle Issues in python/cpython

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.