Reference Counting (memory management) Bugs for `PyList_SetItem`

[wr-web]

Remove incorrect Py_XDECREF/Py_DECREF call for v_obj.

Related Code

Python-project-Scripts/OTHERS/Jarvis/ools/PyQt5/bindings/QAxContainer/qaxbase.sip

Lines 54 to 56 in b0e44c6

	if (PyList_SetItem(a1Wrapper, i, v_obj) < 0)
	{

Python-project-Scripts/OTHERS/Jarvis/ools/PyQt5/bindings/QAxContainer/qaxbase.sip

Lines 93 to 95 in b0e44c6

	if (PyList_SetItem(a1Wrapper, i, v_obj) < 0)
	{

PyList_SetItem Source Code

PyList_SetItem Document

PyList_SetItem Python Forum Discussion

Therefore, whether the function succeeds or fails, it will steal a reference count of the third argument. It must not call Py_XDECREF/Py_DECREF in case of failure.

Similar issue/PR:

PaddlePaddle/Paddle#77447

apache/geaflow#725

[github-actions]

Hi @wr-web! 👋

Thank you for creating an issue in our repository! We appreciate your contribution and will get back to you as soon as possible.