]> granicus.if.org Git - postgresql/commit
Ensure an index that uses a whole-row Var still depends on its table.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 2 Nov 2010 21:15:29 +0000 (17:15 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 2 Nov 2010 21:15:29 +0000 (17:15 -0400)
commit6b1952b07176723b98d8ca0ce3d913bf1360c8af
tree40ecd0d2e67b82e507c57d9f64259545af4097d0
parent2487e8d8c8ca0014d9c02de16d0bdce62da64ad7
Ensure an index that uses a whole-row Var still depends on its table.

We failed to record any dependency on the underlying table for an index
declared like "create index i on t (foo(t.*))".  This would create trouble
if the table were dropped without previously dropping the index.  To fix,
simplify some overly-cute code in index_create(), accepting the possibility
that sometimes the whole-table dependency will be redundant.  Also document
this hazard in dependency.c.  Per report from Kevin Grittner.

In passing, prevent a core dump in pg_get_indexdef() if the index's table
can't be found.  I came across this while experimenting with Kevin's
example.  Not sure it's a real issue when the catalogs aren't corrupt, but
might as well be cautious.

Back-patch to all supported versions.
src/backend/catalog/dependency.c
src/backend/catalog/index.c
src/backend/utils/adt/ruleutils.c