]> granicus.if.org Git - postgresql/commit
Fix failure to set ActiveSnapshot while rewinding a cursor.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 7 May 2014 18:25:17 +0000 (14:25 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 7 May 2014 18:25:17 +0000 (14:25 -0400)
commit022b5f2b228e2d0a658b808340bd32ba904b87f4
tree7607d174dfd2383d462b422d9dce1fb0edbc11bc
parentdfac804a474806692e8e1fc2b65b6bc73666d32d
Fix failure to set ActiveSnapshot while rewinding a cursor.

ActiveSnapshot needs to be set when we call ExecutorRewind because some
plan node types may execute user-defined functions during their ReScan
calls (nodeLimit.c does so, at least).  The wisdom of that is somewhat
debatable, perhaps, but for now the simplest fix is to make sure the
required context is valid.  Failure to do this typically led to a
null-pointer-dereference core dump, though it's possible that in more
complex cases a function could be executed with the wrong snapshot
leading to very subtle misbehavior.

Per report from Leif Jensen.  It's been broken for a long time, so
back-patch to all active branches.
src/backend/tcop/pquery.c
src/test/regress/expected/portals.out
src/test/regress/sql/portals.sql