pgsql: Fix lookup error in extended stats ownership check - Mailing list pgsql-committers

From Tomas Vondra
Subject pgsql: Fix lookup error in extended stats ownership check
Date
Msg-id E1mL6tr-0003WY-Ih@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix lookup error in extended stats ownership check

When an ownership check on extended statistics object failed, the code
was calling aclcheck_error_type to report the failure, which is clearly
wrong, resulting in cache lookup errors. Fix by calling aclcheck_error.

This issue exists since the introduction of extended statistics, so
backpatch all the way back to PostgreSQL 10. It went unnoticed because
there were no tests triggering the error, so add one.

Reported-by: Mark Dilger
Backpatch-through: 10, where extended stats were introduced
Discussion: https://postgr.es/m/1F238937-7CC2-4703-A1B1-6DC225B8978A%40enterprisedb.com

Branch
------
REL_10_STABLE

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

Modified Files
--------------
src/backend/catalog/objectaddress.c     |  3 ++-
src/test/regress/expected/stats_ext.out | 11 +++++++++++
src/test/regress/sql/stats_ext.sql      |  9 +++++++++
3 files changed, 22 insertions(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Tomas Vondra
Date:
Subject: pgsql: Fix lookup error in extended stats ownership check
Next
From: Tomas Vondra
Date:
Subject: pgsql: Rename the role in stats_ext to have regress_ prefix