pgsql: Set scan direction appropriately for SubPlans (bug #15336) - Mailing list pgsql-committers

From Andrew Gierth
Subject pgsql: Set scan direction appropriately for SubPlans (bug #15336)
Date
Msg-id E1fqgia-0000TN-Gy@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Set scan direction appropriately for SubPlans (bug #15336)

When executing a SubPlan in an expression, the EState's direction
field was left alone, resulting in an attempt to execute the subplan
backwards if it was encountered during a backwards scan of a cursor.
Also, though much less likely, it was possible to reach the execution
of an InitPlan while in backwards-scan state.

Repair by saving/restoring estate->es_direction and forcing forward
scan mode in the relevant places.

Backpatch all the way, since this has been broken since 8.3 (prior to
commit c7ff7663e, SubPlans had their own EStates rather than sharing
the parent plan's, so there was no confusion over scan direction).

Per bug #15336 reported by Vladimir Baranoff; analysis and patch by
me, review by Tom Lane.

Discussion: https://postgr.es/m/153449812167.1304.1741624125628126322@wrigleys.postgresql.org

Branch
------
REL_11_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/67b161eae32b0e900f74a2fe0b3f01667ca70850

Modified Files
--------------
src/backend/executor/nodeSubplan.c      | 26 ++++++++++++++++++++++++--
src/test/regress/expected/subselect.out | 17 +++++++++++++++++
src/test/regress/sql/subselect.sql      | 16 ++++++++++++++++
3 files changed, 57 insertions(+), 2 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Fix configure's snprintf test so it exposes HP-UX bug.
Next
From: Peter Eisentraut
Date:
Subject: pgsql: InsertPgAttributeTuple() to set attcacheoff