• Stephen Frost's avatar
    RLS: Fix ALL vs. SELECT+UPDATE policy usage · aa5d3c0b
    Stephen Frost authored
    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.
    aa5d3c0b
rowsecurity.c 24.6 KB