aboutsummaryrefslogtreecommitdiff
path: root/src/backend/commands/functioncmds.c
diff options
context:
space:
mode:
authorNoah Misch <noah@leadboat.com>2013-06-12 19:50:37 -0400
committerNoah Misch <noah@leadboat.com>2013-06-12 19:50:37 -0400
commit813895e4acfa646c6c0405a0a8c5d05637f42865 (patch)
tree435a5ec48158d85c58f357c516e59e777632e466 /src/backend/commands/functioncmds.c
parentfb435f40d5e34f85076a0af56b2f3bf7b86122b8 (diff)
downloadpostgresql-813895e4acfa646c6c0405a0a8c5d05637f42865.tar.gz
postgresql-813895e4acfa646c6c0405a0a8c5d05637f42865.zip
Don't pass oidvector by value.
Since the structure ends with a flexible array, doing so truncates any vector having more than one element. New in 9.3, so no back-patch.
Diffstat (limited to 'src/backend/commands/functioncmds.c')
-rw-r--r--src/backend/commands/functioncmds.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/backend/commands/functioncmds.c b/src/backend/commands/functioncmds.c
index 38187a837c6..c776758b51f 100644
--- a/src/backend/commands/functioncmds.c
+++ b/src/backend/commands/functioncmds.c
@@ -1626,18 +1626,18 @@ DropCastById(Oid castOid)
*/
void
IsThereFunctionInNamespace(const char *proname, int pronargs,
- oidvector proargtypes, Oid nspOid)
+ oidvector *proargtypes, Oid nspOid)
{
/* check for duplicate name (more friendly than unique-index failure) */
if (SearchSysCacheExists3(PROCNAMEARGSNSP,
CStringGetDatum(proname),
- PointerGetDatum(&proargtypes),
+ PointerGetDatum(proargtypes),
ObjectIdGetDatum(nspOid)))
ereport(ERROR,
(errcode(ERRCODE_DUPLICATE_FUNCTION),
errmsg("function %s already exists in schema \"%s\"",
funcname_signature_string(proname, pronargs,
- NIL, proargtypes.values),
+ NIL, proargtypes->values),
get_namespace_name(nspOid))));
}