> On Jun 17, 2021, at 3:39 AM, osumi.takami...@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 scanning pg_index to find indexes associated
with the relation, pg_index is closed and the access share lock released. I
couldn't prove to myself that by the time we use the rd_replidindex field thus
computed that it was safe to assume that the Oid stored there still refers to
an index. The most likely problem would be that the index has since been
dropped in a concurrent transaction, but it also seems just barely possible
that the Oid has been reused and refers to something else, a table perhaps.
The check that I added is not completely bulletproof, because the new object
reusing that Oid could be a different index, and we'd be none the wiser. Do
you think we should do something about that? I felt the checks I put in place
were very cheap and would work in almost all cases. In any event, they seemed
better than no checks, which is what we have now.
—
Mark Dilger
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
