database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-05-12 16:21:30 +03:00
Code

Fix use-after-release mistake in currtid() and currtid2() for views

Browse Source 
This issue has been present since the introduction of this code as of
a3519a2 from 2002, and has been found by buildfarm member prion that
uses RELCACHE_FORCE_RELEASE via the tests introduced recently in
e786be5.

Discussion: https://postgr.es/m/20200601022055.GB4121@paquier.xyz
Backpatch-through: 9.5
This commit is contained in:
Michael Paquier 2020-06-01 14:41:25 +09:00
parent 95e389b3c2
commit 894041eb26
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/backend/utils/adt/tid.c
View File

@ -338,8 +338,13 @@ currtid_for_view(Relation viewrel, ItemPointer tid)
rte = rt_fetch(var->varno, query->rtable); rte = rt_fetch(var->varno, query->rtable);
if (rte) if (rte)
{ {
Datum result;
result = DirectFunctionCall2(currtid_byreloid,
ObjectIdGetDatum(rte->relid),
PointerGetDatum(tid));
table_close(viewrel, AccessShareLock); table_close(viewrel, AccessShareLock);
return DirectFunctionCall2(currtid_byreloid, ObjectIdGetDatum(rte->relid), PointerGetDatum(tid)); return result;
} }
} }
} }