]> granicus.if.org Git - postgresql/commit
Keep exec_simple_check_plan() from thinking "SELECT foo INTO bar" is simple.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 19 Aug 2010 18:10:48 +0000 (18:10 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 19 Aug 2010 18:10:48 +0000 (18:10 +0000)
commitf4b4a46f01eca60f4e7a2827cf13ef8bbff5f1f8
treebb9f4ebbd12a89d570192649367fb91d8683c762
parent3869e9aecbe11fe04b6db4f0da1ace1e0eb7cdd2
Keep exec_simple_check_plan() from thinking "SELECT foo INTO bar" is simple.

It's not clear if this situation can occur in plpgsql other than via the
EXECUTE USING case Heikki illustrated, which I will shortly close off.
However, ignoring the intoClause if it's there is surely wrong, so let's
patch it for safety.

Backpatch to 8.3, which is as far back as this code has a PlannedStmt
to deal with.  There might be another way to make an equivalent test
before that, but since this is just preventing hypothetical bugs,
I'm not going to obsess about it.
src/pl/plpgsql/src/pl_exec.c