pgsql: Prevent inlining a SQL function with multiple OUT parameters. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Prevent inlining a SQL function with multiple OUT parameters.
Date
Msg-id E1PNfeB-0004LS-6n@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Prevent inlining a SQL function with multiple OUT parameters.

There were corner cases in which the planner would attempt to inline such
a function, which would result in a failure at runtime due to loss of
information about exactly what the result record type is.  Fix by disabling
inlining when the function's recorded result type is RECORD.  There might
be some sub-cases where inlining could still be allowed, but this is a
simple and backpatchable fix, so leave refinements for another day.
Per bug #5777 from Nate Carson.

Back-patch to all supported branches.  8.1 happens to avoid a core-dump
here, but it still does the wrong thing.

Branch
------
REL8_3_STABLE

Details
-------
http://git.postgresql.org/gitweb?p=postgresql.git;a=commitdiff;h=0d45e8c560793aa4c4b278a0ce7c002735f37ea5

Modified Files
--------------
src/backend/executor/functions.c         |    5 +++++
src/backend/optimizer/util/clauses.c     |    5 +++++
src/test/regress/expected/rangefuncs.out |   20 ++++++++++++++++++++
src/test/regress/sql/rangefuncs.sql      |   19 +++++++++++++++++++
4 files changed, 49 insertions(+), 0 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Prevent inlining a SQL function with multiple OUT parameters.
Next
From: Heikki Linnakangas
Date:
Subject: pgsql: Refactor the pg_dump zlib code from pg_backup_custom.c to a sepa