* Dean Rasheed (dean.a.rash...@gmail.com) wrote: > On 27 February 2015 at 03:10, Etsuro Fujita <fujita.ets...@lab.ntt.co.jp> > wrote: > > On 2015/02/26 11:38, Stephen Frost wrote: > >> > >> I've pushed an update for this to master and 9.4 and improved the > >> comments and the commit message as discussed. > >> > >> Would be great if you could test and let me know if you run into any > >> issues! > > > > I just spotted a trivial bug in this patch -- in > expand_security_quals() you need to set targetRelation = false inside > the loop, otherwise it will be true for the target relation and all > that follow it. That was why the regression test output from > rls.v4.patch on the other thread wasn't what I expected.
Wow, no, it's done at the entry to the function. I really thought that
was defined and initialized inside the foreach().. That was certainly
my intent.
Will fix, many thanks!
Stephen
signature.asc
Description: Digital signature
