diff options
| author | Christian Heimes <christian@cheimes.de> | 2013-07-01 21:43:09 (GMT) |
|---|---|---|
| committer | Christian Heimes <christian@cheimes.de> | 2013-07-01 21:43:09 (GMT) |
| commit | 1d5b933504c77707792c325a3e147028bdc2dd3a (patch) | |
| tree | d7ab03c48db70df2c045059e8f91b0bed70587e9 /Python/pystate.c | |
| parent | 3126a3d156352167cdbd89bde232842530c9cbbd (diff) | |
| parent | b9dbc7d6e1ad5700ed0084f63215db97a2c9bcbb (diff) | |
| download | cpython-1d5b933504c77707792c325a3e147028bdc2dd3a.zip cpython-1d5b933504c77707792c325a3e147028bdc2dd3a.tar.gz cpython-1d5b933504c77707792c325a3e147028bdc2dd3a.tar.bz2 | |
Issue #18328: Reorder ops in PyThreadState_Delete*() functions. Now the
tstate is first removed from TLS and then deallocated.
CID 1019639 (#1 of 1): Use after free (USE_AFTER_FREE)
use_after_free: Using freed pointer tstate.
Diffstat (limited to 'Python/pystate.c')
| -rw-r--r-- | Python/pystate.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Python/pystate.c b/Python/pystate.c index a8ed138..6609ee9 100644 --- a/Python/pystate.c +++ b/Python/pystate.c @@ -374,11 +374,11 @@ PyThreadState_Delete(PyThreadState *tstate) { if (tstate == _Py_atomic_load_relaxed(&_PyThreadState_Current)) Py_FatalError("PyThreadState_Delete: tstate is still current"); - tstate_delete_common(tstate); #ifdef WITH_THREAD if (autoInterpreterState && PyThread_get_key_value(autoTLSkey) == tstate) PyThread_delete_key_value(autoTLSkey); #endif /* WITH_THREAD */ + tstate_delete_common(tstate); } @@ -392,9 +392,9 @@ PyThreadState_DeleteCurrent() Py_FatalError( "PyThreadState_DeleteCurrent: no current tstate"); _Py_atomic_store_relaxed(&_PyThreadState_Current, NULL); - tstate_delete_common(tstate); if (autoInterpreterState && PyThread_get_key_value(autoTLSkey) == tstate) PyThread_delete_key_value(autoTLSkey); + tstate_delete_common(tstate); PyEval_ReleaseLock(); } #endif /* WITH_THREAD */ |