diff options
| author | Michael Stahl <Michael.Stahl@cib.de> | 2020-01-09 12:41:06 +0100 |
|---|---|---|
| committer | Michael Stahl <michael.stahl@cib.de> | 2020-01-09 13:55:45 +0100 |
| commit | 777ae8101e23a31bdf806c1d09f0c849e47bb8ef (patch) | |
| tree | fbad14ee09ff0e4f0a1b5662eec6cbcf7bf4c5bb /pyuno | |
| parent | f7456777224f95654a179d7320e3e90fe42ce581 (diff) | |
pyuno: remove racy debug check
The use-after-free is easier to trigger than expected; i don't see how
it's possible to check this without a race.
Change-Id: I4afb7066d1bf6b34f4f56aa0f51be0643ae40a66
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/86481
Tested-by: Jenkins
Reviewed-by: Michael Stahl <michael.stahl@cib.de>
Diffstat (limited to 'pyuno')
| -rw-r--r-- | pyuno/source/module/pyuno_runtime.cxx | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/pyuno/source/module/pyuno_runtime.cxx b/pyuno/source/module/pyuno_runtime.cxx index 3e2bb0c6b0a6..823355a900da 100644 --- a/pyuno/source/module/pyuno_runtime.cxx +++ b/pyuno/source/module/pyuno_runtime.cxx @@ -978,13 +978,6 @@ PyThreadAttach::PyThreadAttach( PyInterpreterState *interp) { // note: *may* be called recursively, with PyThreadDetach between - in // that case, don't create *new* PyThreadState but reuse! -#ifndef NDEBUG -#if PY_MAJOR_VERSION == 3 && PY_MINOR_VERSION >= 7 - PyThreadState const*const current = _PyThreadState_UncheckedGet(); - // dereference isn't safe but let's hope it's tolerable for debugging purpose - assert((current == nullptr || current->thread_id != PyThread_get_thread_ident()) && "recursive PyThreadAttach"); -#endif -#endif tstate = PyGILState_GetThisThreadState(); // from TLS, possibly detached if (!tstate) { |