]> granicus.if.org Git - postgresql/commit
Avoid an Assert failure in deconstruct_array() by making get_attstatsslot()
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 9 Jul 2010 22:57:54 +0000 (22:57 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 9 Jul 2010 22:57:54 +0000 (22:57 +0000)
commit22afb5915cea37122231e9255610e72901dd527c
treeae725650389f3ee14dda2e6c96adb95f4b9bc665
parent1f1d13beaba40d092e4c7a337f6f1e01db2a2606
Avoid an Assert failure in deconstruct_array() by making get_attstatsslot()
use the actual element type of the array it's disassembling, rather than
trusting the type OID passed in by its caller.  This is needed because
sometimes the planner passes in a type OID that's only binary-compatible
with the target column's type, rather than being an exact match.  Per an
example from Bernd Helmle.

Possibly we should refactor get_attstatsslot/free_attstatsslot to not expect
the caller to supply type ID data at all, but for now I'll just do the
minimum-change fix.

Back-patch to 7.4.  Bernd's test case only crashes back to 8.0, but since
these subroutines are the same in 7.4, I suspect there may be variant
cases that would crash 7.4 as well.
src/backend/utils/cache/lsyscache.c