Fix crasher bug in array_position(s)
authorAlvaro Herrera <alvherre@alvh.no-ip.org>
Fri, 9 Dec 2016 15:42:17 +0000 (12:42 -0300)
committerAlvaro Herrera <alvherre@alvh.no-ip.org>
Fri, 9 Dec 2016 15:42:17 +0000 (12:42 -0300)
commita73491e5fee88f5db70d69e81fa45060b6ed3682
treea2cca864d4556a42659095986505e1071a640806
parent64bc26f90d342ca343f5ba383a97691a58991204
Fix crasher bug in array_position(s)

array_position and its cousin array_positions were caching the element
type equality function's FmgrInfo without being careful enough to put it
in a long-lived context.  This is obviously broken but it didn't matter
in most cases; only when using arrays of records (involving record_eq)
it becomes a problem.  The fix is to ensure that the type's equality
function's FmgrInfo is cached in the array_position's flinfo->fn_mcxt
rather than the current memory context.

Apart from record types, the only other case that seems complex enough
to possibly cause the same problem are range types.  I didn't find a way
to reproduce the problem with those, so I only include the test case
submitted with the bug report as regression test.

Bug report and patch: Junseok Yang
Discussion: https://postgr.es/m/CAE+byMupUURYiZ6bKYgMZb9pgV1CYAijJGqWj-90W=nS7uEOeA@mail.gmail.com
Backpatch to 9.5, where array_position appeared.
src/backend/utils/adt/array_userfuncs.c
src/test/regress/expected/arrays.out
src/test/regress/sql/arrays.sql