pgsql: Use a separate interpreter for each calling SQL userid in plperl - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Use a separate interpreter for each calling SQL userid in plperl
Date
Msg-id E1P1QcQ-0007C5-R6@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Use a separate interpreter for each calling SQL userid in plperl and pltcl.

There are numerous methods by which a Perl or Tcl function can subvert
the behavior of another such function executed later; for example, by
redefining standard functions or operators called by the target function.
If the target function is SECURITY DEFINER, or is called by such a
function, this means that any ordinary SQL user with Perl or Tcl language
usage rights can do essentially anything with the privileges of the target
function's owner.

To close this security hole, create a separate Perl or Tcl interpreter for
each SQL userid under which plperl or pltcl functions are executed within
a session.  However, all plperlu or pltclu functions run within a session
still share a single interpreter, since they all execute at the trust
level of a database superuser anyway.

Note: this change results in a functionality loss when libperl has been
built without the "multiplicity" option: it's no longer possible to call
plperl functions under different userids in one session, since such a
libperl can't support multiple interpreters in one process.  However, such
a libperl already failed to support concurrent use of plperl and plperlu,
so it's likely that few people use such versions with Postgres.

Security: CVE-2010-3433

Branch
------
REL8_4_STABLE

Details
-------
http://git.postgresql.org/gitweb?p=postgresql.git;a=commitdiff;h=b805be2587972c153dbc047dc2cc74edef92a36d

Modified Files
--------------
doc/src/sgml/installation.sgml |    5 +
doc/src/sgml/plperl.sgml       |   61 ++++--
doc/src/sgml/pltcl.sgml        |   44 +++-
doc/src/sgml/release-7.4.sgml  |   37 +++
doc/src/sgml/release-8.0.sgml  |   37 +++
doc/src/sgml/release-8.1.sgml  |   37 +++
doc/src/sgml/release-8.2.sgml  |   37 +++
doc/src/sgml/release-8.3.sgml  |   37 +++
doc/src/sgml/release-8.4.sgml  |   37 +++
src/pl/plperl/plperl.c         |  560 +++++++++++++++++++++++++---------------
src/pl/tcl/pltcl.c             |  343 ++++++++++++++----------
11 files changed, 852 insertions(+), 383 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Use a separate interpreter for each calling SQL userid in plperl
Next
From: hinoue@pgfoundry.org (User Hinoue)
Date:
Subject: psqlodbc - psqlodbc: Update win64.mak so that gssapi support is