diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2010-07-30 17:57:32 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2010-07-30 17:57:32 +0000 |
| commit | 10c05657d363a185ba433876b9ac99302d67ee0d (patch) | |
| tree | 6d64e92b5ccc5d3cd7d9b15057ad429889c9852a /src/include/parser/parse_func.h | |
| parent | 678e0d92c2d7e3b9648e07fea8d288571091d699 (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.
Completion of back-port of my patch of yesterday.
Diffstat (limited to 'src/include/parser/parse_func.h')
| -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 96a6c263467..f1062cfba73 100644 --- a/src/include/parser/parse_func.h +++ b/src/include/parser/parse_func.h @@ -7,7 +7,7 @@ * Portions Copyright (c) 1996-2003, PostgreSQL Global Development Group * Portions Copyright (c) 1994, Regents of the University of California * - * $Id: parse_func.h,v 1.50 2003/08/04 02:40:14 momjian Exp $ + * $Id: parse_func.h,v 1.50.4.1 2010/07/30 17:57:32 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -78,4 +78,6 @@ extern Oid LookupFuncName(List *funcname, int nargs, const Oid *argtypes, extern Oid LookupFuncNameTypeNames(List *funcname, List *argtypes, bool noError); +extern void check_pg_get_expr_args(ParseState *pstate, Oid fnoid, List *args); + #endif /* PARSE_FUNC_H */ |