From d106f5811cb7d81a7ce45288ef555cab07aaded3 Mon Sep 17 00:00:00 2001 From: Friedemann Kleint Date: Wed, 16 Nov 2022 09:48:56 +0100 Subject: Fix a deadlock when using QMetaObject.invokeMethod(BlockingQueuedConnection) Add allow-thread to the newly introduced functions. Amends aae2f599e8215e49747c9b5dce698d35b913c6c7. Pick-to: 6.4 Fixes: PYSIDE-2121 Task-number: PYSIDE-1898 Change-Id: I2243c2809b275e23ececae0cdd9225db2aaf7225 Reviewed-by: Christian Tismer --- sources/pyside6/PySide6/glue/qtcore.cpp | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/sources/pyside6/PySide6/glue/qtcore.cpp b/sources/pyside6/PySide6/glue/qtcore.cpp index 02ca55a78..c432275b1 100644 --- a/sources/pyside6/PySide6/glue/qtcore.cpp +++ b/sources/pyside6/PySide6/glue/qtcore.cpp @@ -1656,13 +1656,17 @@ QtCoreHelper::QGenericReturnArgumentHolder result(qArgData.metaType, qArgData.da // invokeMethod(QObject *,const char *, QGenericArgument a0, a1, a2 ) // @snippet qmetaobject-invokemethod-arg +PyThreadState *_save = PyEval_SaveThread(); // Py_BEGIN_ALLOW_THREADS const bool result = %CPPSELF.invokeMethod(%1, %2, %3, %4, %5); +PyEval_RestoreThread(_save); // Py_END_ALLOW_THREADS %PYARG_0 = %CONVERTTOPYTHON[bool](result); // @snippet qmetaobject-invokemethod-arg // invokeMethod(QObject *,const char *,Qt::ConnectionType, QGenericArgument a0, a1, a2 ) // @snippet qmetaobject-invokemethod-conn-type-arg +PyThreadState *_save = PyEval_SaveThread(); // Py_BEGIN_ALLOW_THREADS const bool result = %CPPSELF.invokeMethod(%1, %2, %3, %4, %5, %6); +PyEval_RestoreThread(_save); // Py_END_ALLOW_THREADS %PYARG_0 = %CONVERTTOPYTHON[bool](result); // @snippet qmetaobject-invokemethod-conn-type-arg @@ -1674,8 +1678,10 @@ static PyObject *invokeMethodHelper(QObject *obj, const char *member, Qt::Connec const QtCoreHelper::QGenericArgumentHolder &v3) { + PyThreadState *_save = PyEval_SaveThread(); // Py_BEGIN_ALLOW_THREADS const bool callResult = QMetaObject::invokeMethod(obj, member, type, returnArg, v1, v2, v3); + PyEval_RestoreThread(_save); // Py_END_ALLOW_THREADS if (!callResult) { PyErr_Format(PyExc_RuntimeError, "QMetaObject::invokeMethod(): Invocation of %s::%s() failed.", obj->metaObject()->className(), member); -- cgit v1.2.3