diff options
| author | Tom Lane | 2010-07-29 23:16:33 +0000 |
|---|---|---|
| committer | Tom Lane | 2010-07-29 23:16:33 +0000 |
| commit | f223bb7a4175e921c37bb12e9417541655219564 (patch) | |
| tree | 7d7e522d3a7ebcf867380b0cdbf9c1ef81fa8701 /src/include/parser | |
| parent | 5b8bd0529ed5912dae04ffc3383bf4ac43e30083 (diff) | |
Improved version of patch to protect pg_get_expr() against misuse:
look through join alias Vars to avoid breaking join queries, and
move the test to someplace where it will catch more possible ways
of calling a function. We still ought to throw away the whole thing
in favor of a data-type-based solution, but that's not feasible in
the back branches.
This needs to be back-patched further than 9.0, but I don't have time
to do so today. Committing now so that the fix gets into 9.0beta4.
Diffstat (limited to 'src/include/parser')
| -rw-r--r-- | src/include/parser/parse_func.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/include/parser/parse_func.h b/src/include/parser/parse_func.h index 3800f287598..8b2fd94bbaa 100644 --- a/src/include/parser/parse_func.h +++ b/src/include/parser/parse_func.h @@ -7,7 +7,7 @@ * Portions Copyright (c) 1996-2010, PostgreSQL Global Development Group * Portions Copyright (c) 1994, Regents of the University of California * - * $PostgreSQL: pgsql/src/include/parser/parse_func.h,v 1.68 2010/01/02 16:58:07 momjian Exp $ + * $PostgreSQL: pgsql/src/include/parser/parse_func.h,v 1.69 2010/07/29 23:16:33 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -82,4 +82,6 @@ extern Oid LookupFuncNameTypeNames(List *funcname, List *argtypes, extern Oid LookupAggNameTypeNames(List *aggname, List *argtypes, bool noError); +extern void check_pg_get_expr_args(ParseState *pstate, Oid fnoid, List *args); + #endif /* PARSE_FUNC_H */ |