pgsql: Fix privilege checks in pg_stats_ext and pg_stats_ext_exprs. - Mailing list pgsql-committers

From Nathan Bossart
Subject pgsql: Fix privilege checks in pg_stats_ext and pg_stats_ext_exprs.
Date
Msg-id E1s3ytZ-001pYO-6F@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix privilege checks in pg_stats_ext and pg_stats_ext_exprs.

The catalog view pg_stats_ext fails to consider privileges for
expression statistics.  The catalog view pg_stats_ext_exprs fails
to consider privileges and row-level security policies.  To fix,
restrict the data in these views to table owners or roles that
inherit privileges of the table owner.  It may be possible to apply
less restrictive privilege checks in some cases, but that is left
as a future exercise.  Furthermore, for pg_stats_ext_exprs, do not
return data for tables with row-level security enabled, as is
already done for pg_stats_ext.

On the back-branches, a fix-CVE-2024-4317.sql script is provided
that will install into the "share" directory.  This file can be
used to apply the fix to existing clusters.

Bumps catversion on 'master' branch only.

Reported-by: Lukas Fittl
Reviewed-by: Noah Misch, Tomas Vondra, Tom Lane
Security: CVE-2024-4317
Backpatch-through: 14

Branch
------
REL_14_STABLE

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

Modified Files
--------------
doc/src/sgml/catalogs.sgml                |   7 +-
src/backend/catalog/Makefile              |   3 +-
src/backend/catalog/fix-CVE-2024-4317.sql | 115 ++++++++++++++++++++++++++++++
src/backend/catalog/system_views.sql      |  11 ++-
src/test/regress/expected/rules.out       |   8 +--
src/test/regress/expected/stats_ext.out   |  43 +++++++++++
src/test/regress/sql/stats_ext.sql        |  27 +++++++
7 files changed, 197 insertions(+), 17 deletions(-)


pgsql-committers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: pgsql: Revert "Skip .DS_Store files in server side utils"
Next
From: Tom Lane
Date:
Subject: pgsql: Last-minute updates for release notes.