]> granicus.if.org Git - postgresql/commit
Fix permission tests for views/tables proven empty by constraint exclusion.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 1 May 2013 22:26:58 +0000 (18:26 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 1 May 2013 22:26:58 +0000 (18:26 -0400)
commit15b0421002624919c62ae3c6574af2a8452bf6c4
treee2d8cb7fb70ba7c965fbcb838e970e86a3120abc
parent4aed94f1660fb55bc825bf7f3135379dab28eb55
Fix permission tests for views/tables proven empty by constraint exclusion.

A view defined as "select <something> where false" had the curious property
that the system wouldn't check whether users had the privileges necessary
to select from it.  More generally, permissions checks could be skipped
for tables referenced in sub-selects or views that were proven empty by
constraint exclusion (although some quick testing suggests this seldom
happens in cases of practical interest).  This happened because the planner
failed to include rangetable entries for such tables in the finished plan.

This was noticed in connection with erroneous handling of materialized
views, but actually the issue is quite unrelated to matviews.  Therefore,
revert commit 200ba1667b3a8d7a9d559d2f05f83d209c9d8267 in favor of a more
direct test for the real problem.

Back-patch to 9.2 where the bug was introduced (by commit
7741dd6590073719688891898e85f0cb73453159).
src/backend/optimizer/path/allpaths.c
src/backend/optimizer/plan/createplan.c
src/test/regress/expected/privileges.out
src/test/regress/sql/privileges.sql