]> git.ipfire.org Git - thirdparty/postgresql.git/commit
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)
commitc196c610b3391209ad43db22905ef5725dad559c
tree7b4d1f486d7a52a98693e3b278065c21a5196909
parentae0696ed32316d0c0a82d55fd54a21c9947bb181
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