[COMMITTERS] pgsql: Make json{b}_populate_recordset() use the right tupledescriptor - Mailing list pgsql-committers

From Tom Lane
Subject [COMMITTERS] pgsql: Make json{b}_populate_recordset() use the right tupledescriptor
Date
Msg-id E1eBjLa-0002ky-Rg@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Make json{b}_populate_recordset() use the right tuple descriptor.

json{b}_populate_recordset() used the tuple descriptor created from the
query-level AS clause without worrying about whether it matched the actual
input record type.  If it didn't, that would usually result in a crash,
though disclosure of server memory contents seems possible as well, for a
skilled attacker capable of issuing crafted SQL commands.  Instead, use
the query-supplied descriptor only when there is no input tuple to look at,
and otherwise get a tuple descriptor based on the input tuple's own type
marking.  The core code will detect any type mismatch in the latter case.

Michael Paquier and Tom Lane, per a report from David Rowley.
Back-patch to 9.3 where this functionality was introduced.

Security: CVE-2017-15098

Branch
------
REL_10_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/c30f082d2767c22cefb8875dcb1932e5ed338db6

Modified Files
--------------
src/backend/utils/adt/jsonfuncs.c   | 39 ++++++++++++++++++++++++++-----------
src/test/regress/expected/json.out  | 13 +++++++++++++
src/test/regress/expected/jsonb.out | 13 +++++++++++++
src/test/regress/sql/json.sql       |  6 ++++++
src/test/regress/sql/jsonb.sql      |  6 ++++++
5 files changed, 66 insertions(+), 11 deletions(-)


--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

pgsql-committers by date:

Previous
From: Noah Misch
Date:
Subject: [COMMITTERS] pgsql: start-scripts: switch to $PGUSER before opening $PGLOG.
Next
From: Tom Lane
Date:
Subject: [COMMITTERS] pgsql: Last-minute updates for release notes.