Several routines in setrefs.c would crash on array refs

due to lack of check for recursing into a null subexpression.
This commit is contained in:
Tom Lane 1999-04-29 00:20:27 +00:00
parent 89cf9303fd
commit 970583ab4f
1 changed files with 13 additions and 1 deletions

View File

@ -7,7 +7,7 @@
* *
* *
* IDENTIFICATION * IDENTIFICATION
* $Header: /cvsroot/pgsql/src/backend/optimizer/plan/setrefs.c,v 1.42 1999/04/26 00:37:46 tgl Exp $ * $Header: /cvsroot/pgsql/src/backend/optimizer/plan/setrefs.c,v 1.43 1999/04/29 00:20:27 tgl Exp $
* *
*------------------------------------------------------------------------- *-------------------------------------------------------------------------
*/ */
@ -747,6 +747,9 @@ replace_agg_clause(Node *clause, List *subplanTargetList)
List *t; List *t;
List *agg_list = NIL; List *agg_list = NIL;
if (clause == NULL)
return NIL;
if (IsA(clause, Var)) if (IsA(clause, Var))
{ {
TargetEntry *subplanVar; TargetEntry *subplanVar;
@ -864,6 +867,9 @@ del_agg_clause(Node *clause)
{ {
List *t; List *t;
if (clause == NULL)
return clause;
if (IsA(clause, Var)) if (IsA(clause, Var))
return clause; return clause;
else if (is_funcclause(clause)) else if (is_funcclause(clause))
@ -942,6 +948,9 @@ check_having_qual_for_vars(Node *clause, List *targetlist_so_far)
{ {
List *t; List *t;
if (clause == NULL)
return targetlist_so_far;
if (IsA(clause, Var)) if (IsA(clause, Var))
{ {
RelOptInfo tmp_rel; RelOptInfo tmp_rel;
@ -1035,6 +1044,9 @@ check_having_for_ungrouped_vars(Node *clause, List *groupClause)
{ {
List *t; List *t;
if (clause == NULL)
return;
if (IsA(clause, Var)) if (IsA(clause, Var))
{ {
/* Ignore vars elsewhere in the having clause, since the /* Ignore vars elsewhere in the having clause, since the