]> granicus.if.org Git - postgresql/commit
Prevent inlining a SQL function with multiple OUT parameters.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 1 Dec 2010 05:53:39 +0000 (00:53 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 1 Dec 2010 05:53:39 +0000 (00:53 -0500)
commitb0e2092319102c9a749ef4afd5efbdc37fb31b83
treea23a911950428aa73a1e59f4a3403bdc424011ea
parentcfb6ac6ba2fd2c9dc6e27acb6930a155c1685155
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.
src/backend/executor/functions.c
src/backend/optimizer/util/clauses.c
src/test/regress/expected/rangefuncs.out
src/test/regress/sql/rangefuncs.sql