aboutsummaryrefslogtreecommitdiffstats
path: root/sources/pyside2/PySide2/QtWidgets
diff options
context:
space:
mode:
authorAlexandru Croitor <alexandru.croitor@qt.io>2018-02-05 16:19:28 +0100
committerAlexandru Croitor <alexandru.croitor@qt.io>2018-02-14 12:10:55 +0000
commit375a8565e1a0f6e0327d50657ac0f08045012809 (patch)
treea9262bea14f4360e0d2334a08bbafc010b469d83 /sources/pyside2/PySide2/QtWidgets
parent52ac602d7b25dc31e681dcddc0b6dc9452fa4f59 (diff)
Fix usage of PyCFunction_GET_FLAGS in pysidesignal.cpp
Using PyCFunction_GET_FLAGS is unsafe, because it does not check whether the argument given is actually a PyCFunction object. This macro needs to be replaced with the function equivalent PyCFunction_GetFlags for stable ABI patch, but this will cause a crash in PyCFunction_Call because we don't check the return value of PyCFunction_GetFlags to see if it fails. Rather than checking the return value, it is safe to preemptively add a PyCFunction_Check before calling the GetFlags function. This does not modify the logic behind signalCall function. The crashing test was homonymoussignalandmethod_test.py, so no new test is needed. Task-number: PYSIDE-593 Change-Id: Id9ac9c0dec454e8e1ce9516dc68af924372a34a9 Reviewed-by: Christian Tismer <tismer@stackless.com>
Diffstat (limited to 'sources/pyside2/PySide2/QtWidgets')
0 files changed, 0 insertions, 0 deletions