> > Per recent discussion, here is yet another updated version of the 
> > instrumentation patch. Changes:
> > * Added guc option "disable_remote_admin", that disables any write 
> > operations (write, unlink, rename) even for the superuser. Set as 
> > PGC_POSTMASTER so it cannot be changed remotely.
> I was envisioning it as disabling all filesystem access --- 
> read as well as write.  Essentially the abstract concept I 
> want is that with this on, even a superuser cannot use 
> Postgres to get at the underlying operating system.  A name 
> like "enable_filesystem_access" would probably be more appropriate.

Um. I thought the entire argument was about *writing* files. But it
should be easy enough to stick requireRemoteAdmin() to all the

For the long term I was thinking something like "restrict_superuser",
which would disable both read and write, and COPY, and untrusted PL
creation, etc, etc. But that's not for 8.1. 

> Also, as I already said, marking it as PGC_POSTMASTER is 
> simply not adequate security.  Once we have some sort of 
> remote admin feature, I would expect it to support adjustment 
> of even postmaster-level options (this would mean forcing a 
> database restart of course) --- you can hardly say that you 
> have a complete remote admin solution if you can't change 
> shared_buffers or max_connections.

The point is you cannot *enable* it once it is *disabled*. Thus you
cannot *elevate* your privileges. Thus not a security issue.

Yes, if it is enabled, you can remotely disbale it and lock yourself
out. But you can *not* do it the other way around. Once it's off, you
need shell access on the box to edit the file and re-enable it.

Once we have a "real remote admin API", it becomes an argument, and it
will have to be adjusted. But we don't have that today, and I see no
need to create a new guc category just for this. After all, some of
these functions will probably go away completely once we have such an


---------------------------(end of broadcast)---------------------------
TIP 5: don't forget to increase your free space map settings

Reply via email to