diff options
| author | Robert Haas <rhaas@postgresql.org> | 2015-03-11 15:26:43 -0400 |
|---|---|---|
| committer | Robert Haas <rhaas@postgresql.org> | 2015-03-11 15:26:43 -0400 |
| commit | bc93ac12c2544b6b3a68b6cb0282e0828fa14a34 (patch) | |
| tree | c55d425c5973276495a1300d0c9f77f91de10566 /src/backend/optimizer/plan/subselect.c | |
| parent | c6b3c939b7e0f1d35f4ed4996e71420a993810d2 (diff) | |
| download | postgresql-bc93ac12c2544b6b3a68b6cb0282e0828fa14a34.tar.gz postgresql-bc93ac12c2544b6b3a68b6cb0282e0828fa14a34.zip | |
Require non-NULL pstate for all addRangeTableEntryFor* functions.
Per discussion, it's better to have a consistent coding rule here.
Michael Paquier, per a node from Greg Stark referencing an old post
from Tom Lane.
Diffstat (limited to 'src/backend/optimizer/plan/subselect.c')
| -rw-r--r-- | src/backend/optimizer/plan/subselect.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/backend/optimizer/plan/subselect.c b/src/backend/optimizer/plan/subselect.c index 5a1d539e8de..acfd0bcfbe5 100644 --- a/src/backend/optimizer/plan/subselect.c +++ b/src/backend/optimizer/plan/subselect.c @@ -1233,6 +1233,7 @@ convert_ANY_sublink_to_join(PlannerInfo *root, SubLink *sublink, RangeTblRef *rtr; List *subquery_vars; Node *quals; + ParseState *pstate; Assert(sublink->subLinkType == ANY_SUBLINK); @@ -1264,6 +1265,9 @@ convert_ANY_sublink_to_join(PlannerInfo *root, SubLink *sublink, if (contain_volatile_functions(sublink->testexpr)) return NULL; + /* Create a dummy ParseState for addRangeTableEntryForSubquery */ + pstate = make_parsestate(NULL); + /* * Okay, pull up the sub-select into upper range table. * @@ -1272,7 +1276,7 @@ convert_ANY_sublink_to_join(PlannerInfo *root, SubLink *sublink, * below). Therefore this is a lot easier than what pull_up_subqueries has * to go through. */ - rte = addRangeTableEntryForSubquery(NULL, + rte = addRangeTableEntryForSubquery(pstate, subselect, makeAlias("ANY_subquery", NIL), false, |