diff options
author | Peter Eisentraut <peter_e@gmx.net> | 2009-11-03 07:53:58 +0000 |
---|---|---|
committer | Peter Eisentraut <peter_e@gmx.net> | 2009-11-03 07:53:58 +0000 |
commit | 4276fe3a3ba058daceec5a1b64738ead2b70f253 (patch) | |
tree | 4547697ee76c7726b90b3a4abcd802f22039f259 /contrib/rtree_gist/rtree_gist.c | |
parent | 73b4c2b853df82db09d81cb4050b9a80dc326028 (diff) | |
download | postgresql-4276fe3a3ba058daceec5a1b64738ead2b70f253.tar.gz postgresql-4276fe3a3ba058daceec5a1b64738ead2b70f253.zip |
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.
Diffstat (limited to 'contrib/rtree_gist/rtree_gist.c')
0 files changed, 0 insertions, 0 deletions