Re: RLS bug in expanding security quals - Mailing list pgsql-hackers

From Stephen Frost
Subject Re: RLS bug in expanding security quals
Date
Msg-id 20151009024811.GE3685@tamriel.snowman.net
Whole thread Raw
In response to Re: RLS bug in expanding security quals  (Dean Rasheed <dean.a.rasheed@gmail.com>)
List pgsql-hackers
Dean,

* Dean Rasheed (dean.a.rasheed@gmail.com) wrote:
> On 8 October 2015 at 15:05, Dean Rasheed <dean.a.rasheed@gmail.com> wrote:
> > Attached is a simple patch that appears to work, but it needs more
> > testing (and some regression tests).
>
> Here's an updated patch with an extra regression test case that
> triggers the issue.

Thanks!

> I've also updated the function comment for expand_security_quals() to
> better explain the situations where it actually has work to do --
> tables with RLS and updates to auto-updatable security barrier views,
> but not SELECTs from security berrier views. This explains why this
> bug doesn't affect security barrier views (UNION ALL views aren't
> auto-updatable), so only 9.5 and HEAD need to be patched.

Excellent, I definitely like the additional comments.

I plan to do a bit more testing tomorrow morning, but barring any
issues found or concerns raised, I'll push this sometime tomorrow.

Thanks again!

Stephen

pgsql-hackers by date:

Previous
From: Haribabu Kommi
Date:
Subject: Re: RLS bug in expanding security quals
Next
From: Etsuro Fujita
Date:
Subject: Re: Foreign join pushdown vs EvalPlanQual