Re: [PATCH] Fix memory corruption in pg_shdepend.c - Mailing list pgsql-hackers

From Peter Geoghegan
Subject Re: [PATCH] Fix memory corruption in pg_shdepend.c
Date
Msg-id CAH2-WznneOHnZZZVd1+OWc_+DRaFefjum_0fqLvScG+aB=Hk0w@mail.gmail.com
Whole thread Raw
In response to Re: [PATCH] Fix memory corruption in pg_shdepend.c  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
On Wed, Oct 20, 2021 at 8:27 PM Michael Paquier <michael@paquier.xyz> wrote:
> Perhaps.  This means the creation of a new database with shared deps
> in contrib/amcheck/t/.  But is amcheck really a correct target here?
> The fields involved here are an int, some OIDs and a char with a given
> subset of values making them harder to check.  pg_catcheck does checks
> across catalogs, maintaining a mapping list as of its definitions.c.

Users should be able to use pg_amcheck as a high-level corruption
detection tool, which should include any new pg_catcheck style catalog
checking functionality. Whether or not we need to involve
contrib/amcheck itself doesn't seem important to me right now. Offhand
I think that we wouldn't, because as you point out pg_catcheck is a
frontend program that checks multiple databases.

-- 
Peter Geoghegan



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: [PATCH] Fix memory corruption in pg_shdepend.c
Next
From: Amit Kapila
Date:
Subject: Re: pgsql: Document XLOG_INCLUDE_XID a little better