]> granicus.if.org Git - postgresql/commit
Avoid crash in eval_const_expressions if a Param's type changes.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 26 Jul 2018 20:08:45 +0000 (16:08 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 26 Jul 2018 20:08:50 +0000 (16:08 -0400)
commit9207a64e1496d9be8b7f33f2ac30052c130c61c3
tree46cf5b6de15cda5d0fdd052c19c44611a3cf68a2
parented7e319eb41361250ccb19379ef206b4a6d866de
Avoid crash in eval_const_expressions if a Param's type changes.

Since commit 6719b238e it's been possible for the values of plpgsql
record field variables to be exposed to the planner as Params.
(Before that, plpgsql never supplied values for such variables during
planning, so that the problematic code wasn't reached.)  Other places
that touch potentially-type-mutable Params either cope gracefully or
do runtime-test-and-ereport checks that the type is what they expect.
But eval_const_expressions() just had an Assert, meaning that it either
failed the assertion or risked crashes due to using an incompatible
value.

In this case, rather than throwing an ereport immediately, we can just
not perform a const-substitution in case of a mismatch.  This seems
important for the same reason that the Param fetch was speculative:
we might not actually reach this part of the expression at runtime.

Test case will follow in a separate commit.

Patch by me, pursuant to bug report from Andrew Gierth.
Back-patch to v11 where the previous commit appeared.

Discussion: https://postgr.es/m/87wotkfju1.fsf@news-spur.riddles.org.uk
src/backend/optimizer/util/clauses.c