pgsql: Fix oversight in commit b5415e3c2187ab304390524f5ae66b4bd2c58279

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Fix oversight in commit b5415e3c2187ab304390524f5ae66b4bd2c58279
Date: 2018-12-31 17:39:21
Message-ID: E1ge1X7-0003bU-Be@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Fix oversight in commit b5415e3c2187ab304390524f5ae66b4bd2c58279.

While rearranging code in tidpath.c, I overlooked the fact that we ought
to check restriction_is_securely_promotable when trying to use a join
clause as a TID qual. Since tideq itself is leakproof, this wouldn't
really allow any interesting leak AFAICT, but it still seems like we
had better check it.

For consistency with the corresponding logic in indxpath.c, also
check rinfo->pseudoconstant. I'm not sure right now that it's
possible for that to be set in a join clause, but if it were,
a match couldn't be made anyway.

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/b2edbbd02d3070c234225da7b2ec10298652658a

Modified Files
--------------
src/backend/optimizer/path/tidpath.c | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Alvaro Herrera 2018-12-31 17:51:03 pgsql: Remove some useless code
Previous Message Peter Eisentraut 2018-12-30 21:24:55 pgsql: Change "checkpoint starting" message to use "wal"