]> granicus.if.org Git - postgresql/commit
Ensure tableoid reads correctly in EvalPlanQual-manufactured tuples.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 12 Mar 2015 17:38:49 +0000 (13:38 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 12 Mar 2015 17:39:09 +0000 (13:39 -0400)
commit443fd0540e298b621be6748dead1fb444556e0d0
tree72cd18a7dddfad0b93c5415365f888ed4b407ebc
parent26d2c5dc8dc24febee8c3dd472407d7bbdcc245d
Ensure tableoid reads correctly in EvalPlanQual-manufactured tuples.

The ROW_MARK_COPY path in EvalPlanQualFetchRowMarks() was just setting
tableoid to InvalidOid, I think on the assumption that the referenced
RTE must be a subquery or other case without a meaningful OID.  However,
foreign tables also use this code path, and they do have meaningful
table OIDs; so failure to set the tuple field can lead to user-visible
misbehavior.  Fix that by fetching the appropriate OID from the range
table.

There's still an issue about whether CTID can ever have a meaningful
value in this case; at least with postgres_fdw foreign tables, it does.
But that is a different problem that seems to require a significantly
different patch --- it's debatable whether postgres_fdw really wants to
use this code path at all.

Simplified version of a patch by Etsuro Fujita, who also noted the
problem to begin with.  The issue can be demonstrated in all versions
having FDWs, so back-patch to 9.1.
src/backend/executor/execMain.c