Re: reuse sysids security hole?

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: Raw Message | Whole Thread | 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

In response to

Responses

Browse pgsql-hackers by date

  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)