mirror of
https://git.postgresql.org/git/postgresql.git
synced 2024-09-04 06:17:19 +02:00
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
This commit is contained in:
parent
6197d7b538
commit
1fe1a04af8
@ -2476,7 +2476,8 @@ check_object_ownership(Oid roleid, ObjectType objtype, ObjectAddress address,
|
|||||||
break;
|
break;
|
||||||
case OBJECT_STATISTIC_EXT:
|
case OBJECT_STATISTIC_EXT:
|
||||||
if (!pg_statistics_object_ownercheck(address.objectId, roleid))
|
if (!pg_statistics_object_ownercheck(address.objectId, roleid))
|
||||||
aclcheck_error_type(ACLCHECK_NOT_OWNER, address.objectId);
|
aclcheck_error(ACLCHECK_NOT_OWNER, objtype,
|
||||||
|
NameListToString(castNode(List, object)));
|
||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
elog(ERROR, "unrecognized object type: %d",
|
elog(ERROR, "unrecognized object type: %d",
|
||||||
|
@ -54,6 +54,17 @@ DROP TABLE ext_stats_test;
|
|||||||
-- Ensure stats are dropped sanely, and test IF NOT EXISTS while at it
|
-- Ensure stats are dropped sanely, and test IF NOT EXISTS while at it
|
||||||
CREATE TABLE ab1 (a INTEGER, b INTEGER, c INTEGER);
|
CREATE TABLE ab1 (a INTEGER, b INTEGER, c INTEGER);
|
||||||
CREATE STATISTICS IF NOT EXISTS ab1_a_b_stats ON a, b FROM ab1;
|
CREATE STATISTICS IF NOT EXISTS ab1_a_b_stats ON a, b FROM ab1;
|
||||||
|
COMMENT ON STATISTICS ab1_a_b_stats IS 'new comment';
|
||||||
|
CREATE ROLE temp_role;
|
||||||
|
SET SESSION AUTHORIZATION temp_role;
|
||||||
|
COMMENT ON STATISTICS ab1_a_b_stats IS 'changed comment';
|
||||||
|
ERROR: must be owner of statistics object ab1_a_b_stats
|
||||||
|
DROP STATISTICS ab1_a_b_stats;
|
||||||
|
ERROR: must be owner of statistics object ab1_a_b_stats
|
||||||
|
ALTER STATISTICS ab1_a_b_stats RENAME TO ab1_a_b_stats_new;
|
||||||
|
ERROR: must be owner of statistics object ab1_a_b_stats
|
||||||
|
RESET SESSION AUTHORIZATION;
|
||||||
|
DROP ROLE temp_role;
|
||||||
CREATE STATISTICS IF NOT EXISTS ab1_a_b_stats ON a, b FROM ab1;
|
CREATE STATISTICS IF NOT EXISTS ab1_a_b_stats ON a, b FROM ab1;
|
||||||
NOTICE: statistics object "ab1_a_b_stats" already exists, skipping
|
NOTICE: statistics object "ab1_a_b_stats" already exists, skipping
|
||||||
DROP STATISTICS ab1_a_b_stats;
|
DROP STATISTICS ab1_a_b_stats;
|
||||||
|
@ -43,6 +43,15 @@ DROP TABLE ext_stats_test;
|
|||||||
-- Ensure stats are dropped sanely, and test IF NOT EXISTS while at it
|
-- Ensure stats are dropped sanely, and test IF NOT EXISTS while at it
|
||||||
CREATE TABLE ab1 (a INTEGER, b INTEGER, c INTEGER);
|
CREATE TABLE ab1 (a INTEGER, b INTEGER, c INTEGER);
|
||||||
CREATE STATISTICS IF NOT EXISTS ab1_a_b_stats ON a, b FROM ab1;
|
CREATE STATISTICS IF NOT EXISTS ab1_a_b_stats ON a, b FROM ab1;
|
||||||
|
COMMENT ON STATISTICS ab1_a_b_stats IS 'new comment';
|
||||||
|
CREATE ROLE temp_role;
|
||||||
|
SET SESSION AUTHORIZATION temp_role;
|
||||||
|
COMMENT ON STATISTICS ab1_a_b_stats IS 'changed comment';
|
||||||
|
DROP STATISTICS ab1_a_b_stats;
|
||||||
|
ALTER STATISTICS ab1_a_b_stats RENAME TO ab1_a_b_stats_new;
|
||||||
|
RESET SESSION AUTHORIZATION;
|
||||||
|
DROP ROLE temp_role;
|
||||||
|
|
||||||
CREATE STATISTICS IF NOT EXISTS ab1_a_b_stats ON a, b FROM ab1;
|
CREATE STATISTICS IF NOT EXISTS ab1_a_b_stats ON a, b FROM ab1;
|
||||||
DROP STATISTICS ab1_a_b_stats;
|
DROP STATISTICS ab1_a_b_stats;
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user