pgsql: Avoid "you don't own a lock of type ExclusiveLock" in GRANT TABL - Mailing list pgsql-committers

From Noah Misch
Subject pgsql: Avoid "you don't own a lock of type ExclusiveLock" in GRANT TABL
Date
Msg-id E1tFhnq-003WcF-TS@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Avoid "you don't own a lock of type ExclusiveLock" in GRANT TABLESPACE.

This WARNING appeared because SearchSysCacheLocked1() read
cc_relisshared before catcache initialization, when the field is false
unconditionally.  On the basis of reading false there, it constructed a
locktag as though pg_tablespace weren't relisshared.  Only shared
catalogs could be affected, and only GRANT TABLESPACE was affected in
practice.  SearchSysCacheLocked1() callers use one other shared-relation
syscache, DATABASEOID.  DATABASEOID is initialized by the end of
CheckMyDatabase(), making the problem unreachable for pg_database.

Back-patch to v13 (all supported versions).  This has no known impact
before v16, where ExecGrant_common() first appeared.  Earlier branches
avoid trouble by having a separate ExecGrant_Tablespace() that doesn't
use LOCKTAG_TUPLE.  However, leaving this unfixed in v15 could ensnare a
future back-patch of a SearchSysCacheLocked1() call.

Reported by Aya Iwata.

Discussion: https://postgr.es/m/OS7PR01MB11964507B5548245A7EE54E70EA212@OS7PR01MB11964.jpnprd01.prod.outlook.com

Branch
------
REL_14_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/2690a4f5ddf2974c322f48588ff6877ec2caade2

Modified Files
--------------
src/backend/utils/cache/syscache.c        | 15 ++++++++++-----
src/test/regress/input/tablespace.source  |  6 ++++++
src/test/regress/output/tablespace.source |  5 +++++
3 files changed, 21 insertions(+), 5 deletions(-)


pgsql-committers by date:

Previous
From: Nathan Bossart
Date:
Subject: pgsql: pg_dump: Add dumpSchema and dumpData derivative flags.
Next
From: Tom Lane
Date:
Subject: pgsql: Fix NULLIF()'s handling of read-write expanded objects.