On Thu, Jun 12, 2008 at 5:35 PM, Tom Lane <[EMAIL PROTECTED]> wrote:
> "Alex Hunsaker" <[EMAIL PROTECTED]> writes:
>> Yep, I thought about doing the reverse.  Namely Just passing the
>> DropStmt to RemoveRelation(s).  But then all the permission check
>> functions are in utility.c.  Splitting those out seemed to be about
>> the same as splitting out all the ObjectAddress stuff...
> Well, that might actually be a good approach: try to get ProcessUtility
> back down to being just a dispatch switch.  It's pretty much of a wart
> that we're doing any permissions checking in utility.c at all.  Possibly
> those functions should be moved to aclchk.c and then used from
> RemoveRelation(s) and friends, which would stay where they are but
> change API.

Ok Ill work up a patch.  Whats that saying about sticking with your
first instinct?

Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org)
To make changes to your subscription:

Reply via email to