Hi,

On Tue, Sep 7, 2010 at 6:16 AM, Alvaro Herrera
<alvherre(at)commandprompt(dot)com> wrote:
> Excerpts from Jim Nasby's message of jue jun 10 17:54:43 -0400 2010:
>> test_us(at)workbook=# select has_table_privilege( 'public', 'test', 'SELECT' );
>> ERROR:  role "public" does not exist
>
> Here's a patch implementing this idea.

It specially treats only "public" in all lower cases, right?
The pseudo-role name is described as "PUBLIC" (upper) in docs,
but we accept only "public" (lower) as the pseudo-name.

BTW, does the patch need to be back-patched to older versions?
Since they use get_roleid_checked() instead of get_role_oid(), the fix
cannot be applied cleanly to them, though it will be similar codes.

-- 
Itagaki Takahiro

In response to

• Re: Bug / shortcoming in has_*_privilege at 2010-09-06 21:16:57 from Alvaro Herrera

Responses

• Re: Bug / shortcoming in has_*_privilege at 2010-10-13 02:10:05 from Robert Haas
• Re: Bug / shortcoming in has_*_privilege at 2010-10-13 02:45:30 from Alvaro Herrera

pgsql-hackers by date

Next:	From: Robert Haas	Date: 2010-10-13 02:10:05
	Subject: Re: Bug / shortcoming in has_*_privilege
Previous:	From: KaiGai Kohei	Date: 2010-10-13 01:04:52
	Subject: Re: security hook on table creation