diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2002-12-04 20:00:37 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2002-12-04 20:00:37 +0000 |
| commit | 53e2b794f98852351a1f9d20182cb4a1bffda134 (patch) | |
| tree | c800a9a53ae3f8a578c929dd878310a334d7befe /src | |
| parent | 20f2872467cfc5de94f3395d5e1c0466852cbd46 (diff) | |
| download | postgresql-53e2b794f98852351a1f9d20182cb4a1bffda134.tar.gz postgresql-53e2b794f98852351a1f9d20182cb4a1bffda134.zip | |
Repair oversight in recent change of dependency extraction code: when
recursing to handle a join alias var, the context had better be set to
be appropriate to the join var's query level. Per report from Hristo Neshev.
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/catalog/dependency.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/backend/catalog/dependency.c b/src/backend/catalog/dependency.c index d159ceba371..c679c61e12a 100644 --- a/src/backend/catalog/dependency.c +++ b/src/backend/catalog/dependency.c @@ -8,7 +8,7 @@ * Portions Copyright (c) 1994, Regents of the University of California * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/catalog/dependency.c,v 1.13 2002/11/30 21:25:04 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/catalog/dependency.c,v 1.14 2002/12/04 20:00:37 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -789,6 +789,11 @@ find_expr_references_walker(Node *node, else if (rte->rtekind == RTE_JOIN) { /* Scan join output column to add references to join inputs */ + List *save_rtables; + + /* We must make the context appropriate for join's level */ + save_rtables = context->rtables; + context->rtables = rtables; if (var->varattno <= 0 || var->varattno > length(rte->joinaliasvars)) elog(ERROR, "find_expr_references_walker: bogus varattno %d", @@ -796,6 +801,7 @@ find_expr_references_walker(Node *node, find_expr_references_walker((Node *) nth(var->varattno - 1, rte->joinaliasvars), context); + context->rtables = save_rtables; } return false; } |