mirrors/postgres
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

transformExpr() was missing some cases it ought to allow; per report

Browse Source 
from Greg Stark.  Also, twiddle the FuncCall case to not scribble on
the input structure, which was the proximate cause of the problem.
Someday we ought to fix things so that transformExpr() isn't called
on already-transformed trees ...
This commit is contained in:
Tom Lane 2003-02-13 18:29:07 +00:00
parent 8d33f80fc4
commit 5f5da0a6cb
1 changed files with 16 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
src/backend/parser/parse_expr.c
View File

@ -8,7 +8,7 @@
*
*
* IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/parser/parse_expr.c,v 1.144 2003/02/10 04:44:46 tgl Exp $
* $Header: /cvsroot/pgsql/src/backend/parser/parse_expr.c,v 1.145 2003/02/13 18:29:07 tgl Exp $
*
*-------------------------------------------------------------------------
*/
@ -322,15 +322,23 @@ transformExpr(ParseState *pstate, Node *expr)
case T_FuncCall:
{
FuncCall *fn = (FuncCall *) expr;
List *targs;
List *args;
/* transform the list of arguments */
foreach(args, fn->args)
/*
* Transform the list of arguments. We use a shallow
* list copy and then transform-in-place to avoid O(N^2)
* behavior from repeated lappend's.
*/
targs = listCopy(fn->args);
foreach(args, targs)
{
lfirst(args) = transformExpr(pstate,
(Node *) lfirst(args));
}
result = ParseFuncOrColumn(pstate,
fn->funcname,
fn->args,
targs,
fn->agg_star,
fn->agg_distinct,
false);
@ -664,12 +672,15 @@ transformExpr(ParseState *pstate, Node *expr)
* taking a conservative approach, and only accepting node
* types that are demonstrably necessary to accept.
*********************************************/
case T_Expr:
case T_Var:
case T_Const:
case T_Param:
case T_Aggref:
case T_ArrayRef:
case T_FuncExpr:
case T_OpExpr:
case T_DistinctExpr:
case T_BoolExpr:
case T_FieldSelect:
case T_RelabelType:
case T_CoerceToDomain: