pgsql: Fix handling of expanded objects in CoerceToDomain and CASE exec

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Fix handling of expanded objects in CoerceToDomain and CASE exec
Date: 2016-12-22 20:01:57
Message-ID: E1cK9Yr-0003sa-8D@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix handling of expanded objects in CoerceToDomain and CASE execution.

When the input value to a CoerceToDomain expression node is a read-write
expanded datum, we should pass a read-only pointer to any domain CHECK
expressions and then return the original read-write pointer as the
expression result. Previously we were blindly passing the same pointer to
all the consumers of the value, making it possible for a function in CHECK
to modify or even delete the expanded value. (Since a plpgsql function
will absorb a passed-in read-write expanded array as a local variable
value, it will in fact delete the value on exit.)

A similar hazard of passing the same read-write pointer to multiple
consumers exists in domain_check() and in ExecEvalCase, so fix those too.

The fix requires adding MakeExpandedObjectReadOnly calls at the appropriate
places, which is simple enough except that we need to get the data type's
typlen from somewhere. For the domain cases, solve this by redefining
DomainConstraintRef.tcache as okay for callers to access; there wasn't any
reason for the original convention against that, other than not wanting the
API of typcache.c to be any wider than it had to be. For CASE, there's
no good solution except to add a syscache lookup during executor start.

Per bug #14472 from Marcos Castedo. Back-patch to 9.5 where expanded
values were introduced.

Discussion: https://postgr.es/m/15225.1482431619@sss.pgh.pa.us

Branch
------
REL9_5_STABLE

Details
-------
http://git.postgresql.org/pg/commitdiff/c472f2a3353dc4907b54ba90b720310bfb2434eb

Modified Files
--------------
src/backend/executor/execQual.c | 25 ++++++++++++++++++++-----
src/backend/utils/adt/domains.c | 10 ++++++++--
src/include/nodes/execnodes.h | 1 +
src/include/utils/typcache.h | 2 +-
src/test/regress/expected/case.out | 26 ++++++++++++++++++++++++++
src/test/regress/expected/plpgsql.out | 20 ++++++++++++++++++++
src/test/regress/sql/case.sql | 28 ++++++++++++++++++++++++++++
src/test/regress/sql/plpgsql.sql | 23 +++++++++++++++++++++++
8 files changed, 127 insertions(+), 8 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2016-12-22 20:41:59 pgsql: doc: Further speed improvements for HTML XSLT build
Previous Message Andres Freund 2016-12-22 19:33:30 pgsql: Skip checkpoints, archiving on idle systems.