mirrors/cpython
1
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-08-30 05:35:08 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 8

[3.13] gh-117657: Fix TSAN data race in _PyEval_SetTrace assertion (gh-131561) (#131564)

Browse source 
The `sys_tracing_threads` variable should be read inside `LOCK_SETUP()`.
(cherry picked from commit 0de5e0c544)

Co-authored-by: Sam Gross <colesbury@gmail.com>
This commit is contained in:
Miss Islington (bot) 2025-03-21 19:48:40 +01:00 • committed by GitHub
parent 55fb3a75d5
commit 94fbe97bbe
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Python/legacy_tracing.c
View file

@ -596,10 +596,10 @@ _PyEval_SetTrace(PyThreadState *tstate, Py_tracefunc func, PyObject *arg)
if (_PySys_Audit(current_tstate, "sys.settrace", NULL) < 0) { if (_PySys_Audit(current_tstate, "sys.settrace", NULL) < 0) {
return -1; return -1;
} }
assert(tstate->interp->sys_tracing_threads >= 0);
// needs to be decref'd outside of the lock // needs to be decref'd outside of the lock
PyObject *old_traceobj; PyObject *old_traceobj;
LOCK_SETUP(); LOCK_SETUP();
assert(tstate->interp->sys_tracing_threads >= 0);
Py_ssize_t tracing_threads = setup_tracing(tstate, func, arg, &old_traceobj); Py_ssize_t tracing_threads = setup_tracing(tstate, func, arg, &old_traceobj);
UNLOCK_SETUP(); UNLOCK_SETUP();
Py_XDECREF(old_traceobj); Py_XDECREF(old_traceobj);