Fix plpgsql's handling of simple expressions in scrollable cursors.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 21 Mar 2025 15:30:42 +0000 (11:30 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 21 Mar 2025 15:30:42 +0000 (11:30 -0400)
commit7fe312f6092f47724a29e30fb2992bd2bc287687
treea705c01f8e6b876b16f8d9fd381c26f1e89e12e3
parentc5c239e26e3876c05b3a2c5e8989817fd38ceed1
Fix plpgsql's handling of simple expressions in scrollable cursors.

exec_save_simple_expr did not account for the possibility that
standard_planner would stick a Materialize node atop the plan
of even a simple Result, if CURSOR_OPT_SCROLL is set.  This led
to an "unexpected plan node type" error.

This is a very old bug, but it'd only be reached by declaring a
cursor for a "SELECT simple-expression" query and explicitly
marking it scrollable, which is an odd thing to do.  So the lack
of prior reports isn't too surprising.

Bug: #18859
Reported-by: Olleg Samoylov <splarv@ya.ru>
Author: Andrei Lepikhov <lepihov@gmail.com>
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/18859-0d5f28ac99a37059@postgresql.org
Backpatch-through: 13
src/pl/plpgsql/src/expected/plpgsql_simple.out
src/pl/plpgsql/src/pl_exec.c
src/pl/plpgsql/src/sql/plpgsql_simple.sql