projects / users / heikki / postgres.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 971e110) | patch
The previous fix in CVS HEAD and 8.4 for handling the case where a cursor
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>
Mon, 5 Jul 2010 09:27:49 +0000 (09:27 +0000)
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>
Mon, 5 Jul 2010 09:27:49 +0000 (09:27 +0000)
commitc778f898188e931bfefda02c97c37864858f14de
treebeafca84c52487176bc0d3daecefe95bd7aafbbdtree
parent971e11028f67770e3d782cfeb1d4115477ace49acommit | diff
The previous fix in CVS HEAD and 8.4 for handling the case where a cursor
being used in a PL/pgSQL FOR loop is closed was inadequate, as Tom Lane
pointed out. The bug affects FOR statement variants too, because you can
close an implicitly created cursor too by guessing the "<unnamed portal X>"
name created for it.

To fix that, "pin" the portal to prevent it from being dropped while it's
being used in a PL/pgSQL FOR loop. Backpatch all the way to 7.4 which is
the oldest supported version.
src/backend/utils/mmgr/portalmem.c diff | blob | blame | history
src/include/utils/portal.h diff | blob | blame | history
src/pl/plpgsql/src/pl_exec.c diff | blob | blame | history
Heikki's random PostgreSQL patches