python / python/cpython

time.tzset() races on process-global timezone state

Offen
#157,500 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

extension-modules type-bug
Vorherrschende Sprache
Python
Sterne
77.2k
Forks
35.9k
PR-Merge-Kennzahlen
PR-Kennzahlen ausstehend

Beschreibung

Bug report

Bug description:
import time, os, threading
N=8; ITERS=5000
barrier=threading.Barrier(N)
def setter():
    barrier.wait()
    for i in range(ITERS):
        os.environ['TZ']='UTC0' if i&1 else 'EST5EDT'
        time.tzset()
def reader():
    barrier.wait()
    for _ in range(ITERS):
        time.localtime(); 
        try: time.strftime('%Z')
        except Exception: pass
ts=[threading.Thread(target=setter) for _ in range(N//2)]+[threading.Thread(target=reader) for _ in range(N//2)]
[t.start() for t in ts]; [t.join() for t in ts]
print("done tzset")
WARNING: ThreadSanitizer: data race
  Read of size 4 by thread T3:
    #0 strncpy
    #1 get_zone       Modules/timemodule.c:1773
    #2 init_timezone  Modules/timemodule.c:1864
    #3 time_tzset     Modules/timemodule.c:1182
  Previous write of size 8 by thread T1 (malloc, in another time_tzset)
  Location is heap block of size 20 allocated by thread T1
(also: read at tmtotuple Modules/timemodule.c:490 — localtime/gmtime struct_time builder)

(truncated sanitizer output)
CPython versions tested on:

CPython main branch

Operating systems tested on:

Linux

Linked PRs
  • gh-157806

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit Modules/timemodule.c, insbesondere get_zone, init_timezone, time_tzset und dem im Sanitizer-Bericht genannten tmtotuple-Pfad. Reproduziere das Beispiel unter ThreadSanitizer und prüfe den verknüpften PR gh-157806; abgeschlossen ist die Aufgabe, wenn die gemeldeten Race Conditions nicht mehr auftreten und die erwartete Funktionsweise der Zeitzonen-APIs erhalten bleibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
c, python
Bereich
operating-systems
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

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