pgsql: Make executor's SELECT INTO code save and restore original tuple - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Make executor's SELECT INTO code save and restore original tuple
Date
Msg-id E1RiaJA-0008Vq-9T@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Make executor's SELECT INTO code save and restore original tuple receiver.

As previously coded, the QueryDesc's dest pointer was left dangling
(pointing at an already-freed receiver object) after ExecutorEnd.  It's a
bit astonishing that it took us this long to notice, and I'm not sure that
the known problem case with SQL functions is the only one.  Fix it by
saving and restoring the original receiver pointer, which seems the most
bulletproof way of ensuring any related bugs are also covered.

Per bug #6379 from Paul Ramsey.  Back-patch to 8.4 where the current
handling of SELECT INTO was introduced.

Branch
------
master

Details
-------
http://git.postgresql.org/pg/commitdiff/dfd26f9c5f371437f243249025863ea9911aacaa

Modified Files
--------------
src/backend/executor/execMain.c           |   20 +++++++++++++++-----
src/test/regress/expected/select_into.out |   25 +++++++++++++++++++++++++
src/test/regress/sql/select_into.sql      |   15 +++++++++++++++
3 files changed, 55 insertions(+), 5 deletions(-)


pgsql-committers by date:

Previous
From: Michael Meskes
Date:
Subject: pgsql: Made code in ecpg better readable.
Next
From: Tom Lane
Date:
Subject: pgsql: Make executor's SELECT INTO code save and restore original tuple