diff options
author | Michael Stahl <Michael.Stahl@cib.de> | 2020-01-09 12:41:06 +0100 |
---|---|---|
committer | Michael Stahl <michael.stahl@allotropia.de> | 2023-02-23 14:12:56 +0100 |
commit | 41758ebf1103c9e8d039d9dbcdd379ccff50415b (patch) | |
tree | 908da4ca26c2632b717c3de50c836095507e6d02 | |
parent | 7192f01212357201e264105c57708d6c8109dc38 (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>
(cherry picked from commit 777ae8101e23a31bdf806c1d09f0c849e47bb8ef)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/86486
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
(cherry picked from commit c82311303bd26bffb74159b2d1294630830a615a)
-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 6945b1f20cce..c91f73b0b086 100644 --- a/pyuno/source/module/pyuno_runtime.cxx +++ b/pyuno/source/module/pyuno_runtime.cxx @@ -980,13 +980,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) { |