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(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