Fix brin_summarize_new_values() to check index type and ownership.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 26 Dec 2015 17:56:09 +0000 (12:56 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 26 Dec 2015 17:56:09 +0000 (12:56 -0500)
commite10838026b373f01d1de0f4f7ea80a60c30565da
tree7e8b23843316870d57ef37b2418fa665d51331e5
parent2e947ba977eba1c0c31a2981d090b7b1897c49c2
Fix brin_summarize_new_values() to check index type and ownership.

brin_summarize_new_values() did not check that the passed OID was for
an index at all, much less that it was a BRIN index, and would fail in
obscure ways if it wasn't (possibly damaging data first?).  It also
lacked any permissions test; by analogy to VACUUM, we should only allow
the table's owner to summarize.

Noted by Jeff Janes, fix by Michael Paquier and me
src/backend/access/brin/brin.c
src/test/regress/expected/brin.out
src/test/regress/sql/brin.sql