From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com> |
Cc: | Josh Kupershmidt <schmiddy(at)gmail(dot)com>, Simon Riggs <simon(at)2ndquadrant(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [GENERAL] column-level update privs + lock table |
Date: | 2010-11-29 01:43:34 |
Message-ID: | AANLkTi=0_m2x0FXLGOMO-SVYw-mBLwMzTXyUfgpt-bw5@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general pgsql-hackers |
2010/11/28 KaiGai Kohei <kaigai(at)ak(dot)jp(dot)nec(dot)com>:
>> I'm not totally convinced that this is the correct behavior. It seems
>> a bit surprising that UPDATE privilege on a single column is enough to
>> lock out all SELECT activity from the table. It's actually a bit
>> surprising that even full-table UPDATE privileges are enough to do
>> this, but this change would allow people to block access to data they
>> can neither see nor modify. That seems counterintuitive, if not a
>> security hole.
>>
> In my understanding, UPDATE privilege on a single column also allows
> lock out concurrent updating even if this query tries to update rows
> partially.
> Therefore, the current code considers UPDATE privilege on a single
> column is enough to lock out the table. Right?
Against concurrent updates and deletes, yes. Against inserts that
don't involve potential unique-index conflicts, and against selects,
no.
> My comment was from a standpoint which wants consistent behavior
> between SELECT ... FOR and LOCK command.
Again, nothing about this makes those consistent.
> If we concerned about this
> behavior, ExecCheckRTEPerms() might be a place where we also should fix.
I don't understand what you're getting at here.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Scott Marlowe | 2010-11-29 01:54:14 | Re: ERROR: xlog flush request 17/4D6C2720 is not satisfied |
Previous Message | Vick Khera | 2010-11-29 00:46:43 | Re: ERROR: xlog flush request 17/4D6C2720 is not satisfied |
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2010-11-29 01:44:38 | Re: Feature request: INSERT .... ON DUPLICATE UPDATE .... |
Previous Message | Robert Haas | 2010-11-29 01:40:08 | Re: Patch to add a primary key using an existing index |