diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2012-09-09 20:33:06 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2012-09-09 20:33:06 -0400 |
| commit | 0952811c86ad64d016d91ced0d8f2a78ce3acda2 (patch) | |
| tree | 9796c900e7e6ede8f3c9c977d51b80715eec6987 /src/include/optimizer/geqo_selection.h | |
| parent | 25b6df1e35190270b2cd04fd3beec7fed8441829 (diff) | |
Make plperl safe against functions that are redefined while running.
validate_plperl_function() supposed that it could free an old
plperl_proc_desc struct immediately upon detecting that it was stale.
However, if a plperl function is called recursively, this could result
in deleting the struct out from under an outer invocation, leading to
misbehavior or crashes. Add a simple reference-count mechanism to
ensure that such structs are freed only when the last reference goes
away.
Per investigation of bug #7516 from Marko Tiikkaja. I am not certain
that this error explains his report, because he says he didn't have
any recursive calls --- but it's hard to see how else it could have
crashed right there. In any case, this definitely fixes some problems
in the area.
Back-patch to all active branches.
Diffstat (limited to 'src/include/optimizer/geqo_selection.h')
0 files changed, 0 insertions, 0 deletions
