]> granicus.if.org Git - postgresql/commit
Fix obscure segfault condition in PL/Python
authorPeter Eisentraut <peter_e@gmx.net>
Tue, 3 Nov 2009 08:12:46 +0000 (08:12 +0000)
committerPeter Eisentraut <peter_e@gmx.net>
Tue, 3 Nov 2009 08:12:46 +0000 (08:12 +0000)
commit1ef3f3cde3d2c5ff233209be01f58825a84348ab
treefc24b3b8ca90751e6eef5c6cb756e8ebc64a6969
parent3a2e45720429cc4328ba1956478ac3d5140bf282
Fix obscure segfault condition in PL/Python

In PLy_output(), when the elog() call in the TRY branch throws an exception
(this can happen when a statement timeout kicks in, for example), the
PyErr_SetString() call in the CATCH branch can cause a segfault, because the
Py_XDECREF(so) call before it releases memory that is still used by the sv
variable that PyErr_SetString() uses as argument, because sv points into
memory owned by so.

Backpatched back to 8.0, where this code was introduced.

I also threw in a couple of volatile declarations for variables that are used
before and after the TRY.  I don't think they caused the crash that I
observed, but they could become issues.
src/pl/plpython/plpython.c