aboutsummaryrefslogtreecommitdiff
path: root/src/backend/tcop/postgres.c
diff options
context:
space:
mode:
authorNoah Misch <noah@leadboat.com>2024-11-25 14:42:35 -0800
committerNoah Misch <noah@leadboat.com>2024-11-25 14:42:39 -0800
commit941e0c0df2bcdeb63bd822a1c46818fa9d7d8599 (patch)
treecf44b7e27082f834ffd8e1b8a1a399727dc16d9c /src/backend/tcop/postgres.c
parent851c6ff18fd359bebd97b7f2b73bb46981a1a8d2 (diff)
downloadpostgresql-941e0c0df2bcdeb63bd822a1c46818fa9d7d8599.tar.gz
postgresql-941e0c0df2bcdeb63bd822a1c46818fa9d7d8599.zip
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
Diffstat (limited to 'src/backend/tcop/postgres.c')
0 files changed, 0 insertions, 0 deletions