aboutsummaryrefslogtreecommitdiff
path: root/src/backend
diff options
context:
space:
mode:
authorItagaki Takahiro <itagaki.takahiro@gmail.com>2010-05-11 04:56:37 +0000
committerItagaki Takahiro <itagaki.takahiro@gmail.com>2010-05-11 04:56:37 +0000
commit871a6086b1919512824f96c2278ed70de546dfc3 (patch)
treecf0e597ccedd98175ae80979bbd3999c99ab60d4 /src/backend
parente0ed3d2aa56b52f8ee7e74392a4096eb10d5b4b9 (diff)
downloadpostgresql-871a6086b1919512824f96c2278ed70de546dfc3.tar.gz
postgresql-871a6086b1919512824f96c2278ed70de546dfc3.zip
Set per-function GUC settings during validating the function.
Now validators work properly even when the settings contain parameters that affect behavior of the function, like search_path. Reported by Erwin Brandstetter.
Diffstat (limited to 'src/backend')
-rw-r--r--src/backend/catalog/pg_proc.c22
1 files changed, 21 insertions, 1 deletions
diff --git a/src/backend/catalog/pg_proc.c b/src/backend/catalog/pg_proc.c
index 18e8b28627f..fc0fc3c54af 100644
--- a/src/backend/catalog/pg_proc.c
+++ b/src/backend/catalog/pg_proc.c
@@ -8,7 +8,7 @@
*
*
* IDENTIFICATION
- * $PostgreSQL: pgsql/src/backend/catalog/pg_proc.c,v 1.164.2.3 2010/03/19 22:54:49 tgl Exp $
+ * $PostgreSQL: pgsql/src/backend/catalog/pg_proc.c,v 1.164.2.4 2010/05/11 04:56:37 itagaki Exp $
*
*-------------------------------------------------------------------------
*/
@@ -556,9 +556,29 @@ ProcedureCreate(const char *procedureName,
/* Verify function body */
if (OidIsValid(languageValidator))
{
+ ArrayType *set_items;
+ int save_nestlevel;
+
/* Advance command counter so new tuple can be seen by validator */
CommandCounterIncrement();
+
+ /* Set per-function configuration parameters */
+ set_items = (ArrayType *) DatumGetPointer(proconfig);
+ if (set_items) /* Need a new GUC nesting level */
+ {
+ save_nestlevel = NewGUCNestLevel();
+ ProcessGUCArray(set_items,
+ (superuser() ? PGC_SUSET : PGC_USERSET),
+ PGC_S_SESSION,
+ GUC_ACTION_SAVE);
+ }
+ else
+ save_nestlevel = 0; /* keep compiler quiet */
+
OidFunctionCall1(languageValidator, ObjectIdGetDatum(retval));
+
+ if (set_items)
+ AtEOXact_GUC(true, save_nestlevel);
}
return retval;