| From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> | 
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> | 
| Cc: | Gavin Sherry <swm(at)linuxworld(dot)com(dot)au>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Postgresql Hackers <pgsql-hackers(at)postgresql(dot)org> | 
| Subject: | Re: reuse sysids security hole? | 
| Date: | 2003-08-12 19:20:40 | 
| Message-ID: | 200308121920.h7CJKeO11503@candle.pha.pa.us | 
| Views: | Whole Thread | Raw Message | Download mbox | Resend email | 
| Thread: | |
| Lists: | pgsql-hackers | 
Can I have a TODO for this?
---------------------------------------------------------------------------
Tom Lane wrote:
> Gavin Sherry <swm(at)linuxworld(dot)com(dot)au> writes:
> > On Tue, 12 Aug 2003, Andrew Dunstan wrote:
> >> Is this a security hole? Looks like one to me. Would it be better to use 
> >> a sequence generator for sysids instead of using max+1 on the user 
> >> table? Or else store the last sysid used somewhere?
> 
> > This issue has been discussed before and it was agreed that since most
> > UNIX systems will behave in the same way, there's no way to know. Also, it
> > is not possible for a given database to know the max(sysid) of pg_user in
> > another database.
> 
> You forget that pg_shadow is a shared (cluster-wide) table.
> 
> I believe we could make a shared sequence object, too, if we wanted to
> go the sequence route.
> 
> Right at the moment I like both ideas: a shared sequence to generate new
> sysids, and don't ever delete pg_shadow rows.  One attraction of the
> sequence generator is that scans over pg_shadow could get rather tedious
> if we follow the latter policy.  But with a sequence, CREATE USER
> wouldn't need to do a scan.
> 
> Something else that should be factored into any redesign of pg_shadow is
> the notion of combining users and groups, at least to the extent of
> having a common sysid space for both.  See discussion started by Peter
> a month or two back (I think thread title mentioned "roles").
> 
> 			regards, tom lane
> 
> ---------------------------(end of broadcast)---------------------------
> TIP 5: Have you checked our extensive FAQ?
> 
>                http://www.postgresql.org/docs/faqs/FAQ.html
> 
-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman(at)candle(dot)pha(dot)pa(dot)us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2003-08-12 19:36:07 | Parsing speed (was Re: pgstats_initstats() cost) | 
| Previous Message | Andrew Dunstan | 2003-08-12 18:57:02 | Re: dropping a user causes pain (#2) |