python / python/cpython

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

Abierto
#157,500 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

extension-modules type-bug
Lenguaje dominante
Python
Estrellas
77.2k
Forks
35.9k
Métricas de merge de PR
Métricas de PR pendientes

Descripción

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

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Empieza por Modules/timemodule.c, especialmente get_zone, init_timezone, time_tzset y la ruta tmtotuple mencionada en el informe del sanitizer. Reproduce el ejemplo con ThreadSanitizer y revisa el PR vinculado gh-157806; se considera terminado cuando las condiciones de carrera indicadas ya no se produzcan y las APIs de zona horaria conserven su comportamiento esperado.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
c, python
Área
operating-systems
Tipo de issue
Error
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.