pgsql: Don't access catalogs to validate GUCs when not connected to a D

From: Andres Freund <andres(at)anarazel(dot)de>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Don't access catalogs to validate GUCs when not connected to a D
Date: 2019-06-11 07:13:06
Message-ID: E1haaxu-0000HW-Oe@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Don't access catalogs to validate GUCs when not connected to a DB.

Vignesh found this bug in the check function for
default_table_access_method's check hook, but that was just copied
from older GUCs. Investigation by Michael and me then found the bug in
further places.

When not connected to a database (e.g. in a walsender connection), we
cannot perform (most) GUC checks that need database access. Even when
only shared tables are needed, unless they're
nailed (c.f. RelationCacheInitializePhase2()), they cannot be accessed
without pg_class etc. being present.

Fix by extending the existing IsTransactionState() checks to also
check for MyDatabaseOid.

Reported-By: Vignesh C, Michael Paquier, Andres Freund
Author: Vignesh C, Andres Freund
Discussion: https://postgr.es/m/CALDaNm1KXK9gbZfY-p_peRFm_XrBh1OwQO1Kk6Gig0c0fVZ2uw%40mail.gmail.com
Backpatch: 9.4-

Branch
------
master

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

Modified Files
--------------
src/backend/access/table/tableamapi.c | 8 +++++---
src/backend/commands/tablespace.c | 14 ++++++++------
src/backend/utils/cache/ts_cache.c | 8 +++++---
3 files changed, 18 insertions(+), 12 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Andres Freund 2019-06-11 07:13:08 pgsql: Don't access catalogs to validate GUCs when not connected to a D
Previous Message Etsuro Fujita 2019-06-11 04:43:59 pgsql: postgres_fdw: Reorder C includes.