At Tue, 07 Jun 2022 23:04:52 -0400, Tom Lane <t...@sss.pgh.pa.us> wrote in 
> Laurenz Albe <laurenz.a...@cybertec.at> writes:
> > On Wed, 2022-06-08 at 11:12 +0900, Kyotaro Horiguchi wrote:
> > RangeTblEntry *rte = root->simple_rte_array[i];
> > aclcheck_error(ACLCHECK_NO_PRIV,
> >    get_relkind_objtype(rte->relkind),
> >    get_rel_name(rte->relid));
> 
> I think it's completely inappropriate for FDWs to be taking it on
> themselves to inject privilege checks.  The system design is that
> that is checked at executor start; not before, not after.

Ah, yes.  It's not good that checking it at multiple stages, and the
only one place should be executor start.

regards.

-- 
Kyotaro Horiguchi
NTT Open Source Software Center


Reply via email to