]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Make standard maintenance operations (including VACUUM, ANALYZE, REINDEX,
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 3 Jan 2008 21:25:34 +0000 (21:25 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 3 Jan 2008 21:25:34 +0000 (21:25 +0000)
commit230d5cfc4739bc4963c8d67e4a7cd84abe53ef93
tree0fa8d138ed251e873bd55a4e7b402fbdd9f57488
parent0776cb2116f4eaec743f1e304c1255c318a20d1f
Make standard maintenance operations (including VACUUM, ANALYZE, REINDEX,
and CLUSTER) execute as the table owner rather than the calling user, using
the same privilege-switching mechanism already used for SECURITY DEFINER
functions.  The purpose of this change is to ensure that user-defined
functions used in index definitions cannot acquire the privileges of a
superuser account that is performing routine maintenance.  While a function
used in an index is supposed to be IMMUTABLE and thus not able to do anything
very interesting, there are several easy ways around that restriction; and
even if we could plug them all, there would remain a risk of reading sensitive
information and broadcasting it through a covert channel such as CPU usage.

To prevent bypassing this security measure, execution of SET SESSION
AUTHORIZATION and SET ROLE is now forbidden within a SECURITY DEFINER context.

Thanks to Itagaki Takahiro for reporting this vulnerability.

Security: CVE-2007-6600
doc/src/sgml/ref/set_session_auth.sgml
src/backend/access/transam/xact.c
src/backend/catalog/index.c
src/backend/commands/schemacmds.c
src/backend/commands/vacuum.c
src/backend/commands/variable.c
src/backend/utils/adt/ri_triggers.c
src/backend/utils/fmgr/fmgr.c
src/backend/utils/init/miscinit.c
src/include/miscadmin.h