]> granicus.if.org Git - postgresql/commit
RLS: Fix ALL vs. SELECT+UPDATE policy usage
authorStephen Frost <sfrost@snowman.net>
Sun, 7 May 2017 01:46:35 +0000 (21:46 -0400)
committerStephen Frost <sfrost@snowman.net>
Sun, 7 May 2017 01:46:35 +0000 (21:46 -0400)
commitaa5d3c0b3fb906dfa910b0ca6f75ab701b2f1c09
treee8d56ea23d0da85ee64c5bb6a0affb2da02ca334
parentb58c433ef90be2f9752cd54561c07dae87e3819c
RLS: Fix ALL vs. SELECT+UPDATE policy usage

When we add the SELECT-privilege based policies to the RLS with check
options (such as for an UPDATE statement, or when we have INSERT ...
RETURNING), we need to be sure and use the 'USING' case if the policy is
actually an 'ALL' policy (which could have both a USING clause and an
independent WITH CHECK clause).

This could result in policies acting differently when built using ALL
(when the ALL had both USING and WITH CHECK clauses) and when building
the policies independently as SELECT and UPDATE policies.

Fix this by adding an explicit boolean to add_with_check_options() to
indicate when the USING policy should be used, even if the policy has
both USING and WITH CHECK policies on it.

Reported by: Rod Taylor

Back-patch to 9.5 where RLS was introduced.
src/backend/rewrite/rowsecurity.c