Re: Fix for segfault in logical replication on master - Mailing list pgsql-hackers

From Mark Dilger
Subject Re: Fix for segfault in logical replication on master
Date
Msg-id FCF0D585-0B9E-458E-82FC-11AAB0BFD1A0@enterprisedb.com
Whole thread Raw
In response to RE: Fix for segfault in logical replication on master  ("osumi.takamichi@fujitsu.com" <osumi.takamichi@fujitsu.com>)
Responses Re: Fix for segfault in logical replication on master
List pgsql-hackers

> On Jun 17, 2021, at 3:39 AM, osumi.takamichi@fujitsu.com wrote:
>
> For the 1st check, isn't it better to use RelationIsValid() ?

Yes, you are right.

> Additionally, In what kind of actual scenario, did you think that
> we come to the part to "log a complaint" ?

The way that RelationGetIndexList assigns rd_replidindex to the Relation seems to lack sufficient locking.  After
scanningpg_index to find indexes associated with the relation, pg_index is closed and the access share lock released.
Icouldn't prove to myself that by the time we use the rd_replidindex field thus computed that it was safe to assume
thatthe Oid stored there still refers to an index.  The most likely problem would be that the index has since been
droppedin a concurrent transaction, but it also seems just barely possible that the Oid has been reused and refers to
somethingelse, a table perhaps.  The check that I added is not completely bulletproof, because the new object reusing
thatOid could be a different index, and we'd be none the wiser.  Do you think we should do something about that?  I
feltthe checks I put in place were very cheap and would work in almost all cases.  In any event, they seemed better
thanno checks, which is what we have now. 

—
Mark Dilger
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company






pgsql-hackers by date:

Previous
From: Dilip Kumar
Date:
Subject: Re: [Proposal] Fully WAL logged CREATE DATABASE - No Checkpoints
Next
From: Tom Lane
Date:
Subject: Re: Add version macro to libpq-fe.h