Re: Updated instrumentation patch

Magnus Hagander wrote:
> > > > I think any secure solution is going to have to block all write
> > > > access to postgresql.conf, and that includes all the COPY
> > TO and all
> > > > the untrusted languages.
> > >
> > > Exactly. But we won't get that for 8.1. So for now, we
> > block all write
> > > access through *new* functions, per the "let's at least not
> > add more
> > > security holes" rule.
> >
> > As far as I know, the only new functionality the patch adds
> > _over_ copy is the ability to write nulls, and rename/unlink.
> >  Should we just throw an error when writing null bytes?
>
> Um. Yes. This patch goes one step further and allows you to block the
> writing of *any* file using these functions. The question is wether that
> one step further is far enough..

I am thinking we can just block null byte writes and say it is the same
as COPY, which we have always used.

--
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073