python / python/cpython

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

未关闭
#157,500 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

extension-modules type-bug
主要语言
Python
星标
77.2k
派生
35.9k
PR 合并指标
PR 指标待抓取

描述

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

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

调研方向

从 Modules/timemodule.c 开始,重点检查 get_zone、init_timezone、time_tzset,以及 sanitizer 报告中所述的 tmtotuple 路径。在 ThreadSanitizer 下重现该示例,并审查关联的 PR gh-157806;当报告的竞态不再发生且时区 API 保持预期行为时,即视为完成。

由索引模型根据 Issue 内容生成。

评估

技术栈
c, python
领域
operating-systems
Issue 类型
缺陷
难度
4/5
预计耗时
3-5 天
活跃度
停滞
描述清晰度
基本清楚
新手友好度
25/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。