Re: Reworks of DML permission checks

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>
Cc: Stephen Frost <sfrost(at)snowman(dot)net>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Reworks of DML permission checks
Date: 2010-07-22 00:48:59
Message-ID: AANLkTini0FGYED8pS4peJ9=PR1Ky_oQGcvi_TfK+1Je3@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

2010/7/19 KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>:
> The attached patch is the revised one.
>
> * It was rebased to the latest git HEAD.
> * Prototype of ExecCheckRTEPerms() was changed; it become to return
>  a bool value to inform the caller its access control decision, and
>  its 'ereport_on_violation' argument has gone.
> * ExecCheckRTPerms() calls aclcheck_error() when ExecCheckRTEPerms()
>  returned false, and 'ereport_on_violation' is true.
> * Add '#include "executor/executor.h"' on the ri_triggers.c.

Committed with some changes to the comments.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise Postgres Company

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Stephen Frost 2010-07-22 01:02:15 Re: ExecutorCheckPerms() hook
Previous Message Simon Riggs 2010-07-22 00:43:01 Re: Add column if not exists (CINE)