Re: [HACKERS] less privileged pl install

From: Jeremy Drake <pgsql(at)jdrake(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Martijn van Oosterhout <kleptog(at)svana(dot)org>, Stefan Kaltenbrunner <stefan(at)kaltenbrunner(dot)cc>, Peter Eisentraut <peter_e(at)gmx(dot)net>, PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org>
Subject: Re: [HACKERS] less privileged pl install
Date: 2007-01-27 06:47:07
Message-ID: Pine.BSO.4.64.0701262246080.23712@resin.csoft.net
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

On Fri, 26 Jan 2007, Jeremy Drake wrote:

> On Sat, 27 Jan 2007, Tom Lane wrote:
>
> > I'd go with GetUserId() in the cases where you're not explicitly
> > assigning ownership to the datdba role. AFAIR the assumption that
> > languages are owned by BOOTSTRAP_SUPERUSERID was just a kluge to use in
> > some bits of code that had to have a notion of a specific owner. Now
> > in reality every superuser has the same privileges as every other one,
> > and so it doesn't matter much which one you use, but we might as well
> > record who actually did the deed.
>
> Here is a version of the patch which does this. Very minor change from
> the last version...

Oops, was missing an include which resulted in an implicitly defined
function warning. This version fixes it...

--
Anybody who doesn't cut his speed at the sight of a police car is
probably parked.

Attachment Content-Type Size
language_priv_changes6.patch text/plain 26.5 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Henry B. Hotz 2007-01-27 09:31:25 Re: 10 weeks to feature freeze (Pending Work)
Previous Message Jeremy Drake 2007-01-27 06:41:17 Re: [HACKERS] less privileged pl install

Browse pgsql-patches by date

  From Date Subject
Next Message Peter Eisentraut 2007-01-27 09:46:11 Re: Change draft gmake control
Previous Message Jeremy Drake 2007-01-27 06:41:17 Re: [HACKERS] less privileged pl install