Re: weird error message in sepgsql

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Peter Eisentraut <peter_e(at)gmx(dot)net>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: weird error message in sepgsql
Date: 2012-05-22 15:30:43
Message-ID: CA+TgmoZn87MU2Yvk6nFgCNAYqWUBMN+cMg3r1zqm_1vf3h9-0A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, May 20, 2012 at 11:28 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
>> I found this in contrib/sepgsql/expected/label.out:
>> SECURITY LABEL ON COLUMN t2
>>     IS 'system_u:object_r:sepgsql_ro_table_t:s0';   -- be failed
>> ERROR:  improper relation name (too many dotted names): <nothing>
>
>> Contrast with:
>
>> SECURITY LABEL ON COLUMN t2.b
>>     IS 'system_u:object_r:sepgsql_ro_table_t:s0';   -- ok
>
>> I guess what's happening is that it's calling makeRangeVarFromNameList()
>> with a list of length zero.
>
>> We should either fix the SECURITY LABEL command to catch that case
>> beforehand, or fix makeRangeVarFromNameList() to give a proper error
>> message, or both.
>
> I think the appropriate error message is probably along the lines of
> "column name must be qualified", and it's hard to justify having
> makeRangeVarFromNameList emit such a thing.  So this is the fault
> of the calling code.

Fixed and back-patched to 9.1. Thanks for the report.

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

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Susanne Ebrecht 2012-05-22 15:36:49 Re: Changing the concept of a DATABASE
Previous Message Tom Lane 2012-05-22 14:25:17 Re: Readme of Buffer Management seems to have wrong sentence