NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit52cdbdc

committed

Fix thinkos in LookupFuncName() for function name lookups

This could trigger valgrind failures when doing ambiguous function namelookups when no arguments are provided by the caller. The problem hasbeen introduced inaefeb68, so backpatch to v10. HEAD is fine thanks tothe refactoring done inbfb456c.Reported-by: Alexander LakhinAuthor: Alexander Lakhin, Michael PaquierDiscussion:https://postgr.es/m/3d068be5-f617-a5ee-99f6-458a407bfd65@gmail.comBackpatch-through: 10

1 parent2839bf3 commit52cdbdcCopy full SHA for 52cdbdc

File tree

1 file changed

-2

lines changed

src/backend/parser
- parse_func.c

1 file changed

-2

lines changed

`‎src/backend/parser/parse_func.c`

Lines changed: 8 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -2059,9 +2059,10 @@ LookupFuncName(List funcname, int nargs, const Oid argtypes, bool noError)`
`2059`	`2059`	`errmsg("function name \"%s\" is not unique",`
`2060`	`2060`	`NameListToString(funcname)),`
`2061`	`2061`	`errhint("Specify the argument list to select the function unambiguously.")));`
	`2062`	`+returnInvalidOid;`
`2062`	`2063`	`}`
`2063`		`-else`
`2064`		`-returnclist->oid;`
	`2064`	`+/* Otherwise return the match */`
	`2065`	`+returnclist->oid;`
`2065`	`2066`	`}`
`2066`	`2067`	`else`
`2067`	`2068`	`{`
`@@ -2070,9 +2071,14 @@ LookupFuncName(List funcname, int nargs, const Oid argtypes, bool noError)`
`2070`	`2071`	`(errcode(ERRCODE_UNDEFINED_FUNCTION),`
`2071`	`2072`	`errmsg("could not find a function named \"%s\"",`
`2072`	`2073`	`NameListToString(funcname))));`
	`2074`	`+returnInvalidOid;`
`2073`	`2075`	`}`
`2074`	`2076`	`}`
`2075`	`2077`
	`2078`	`+/*`
	`2079`	`+ * Otherwise, look for a match to the arg types. FuncnameGetCandidates`
	`2080`	`+ * has ensured that there's at most one match in the returned list.`
	`2081`	`+ */`
`2076`	`2082`	`while (clist)`
`2077`	`2083`	`{`
`2078`	`2084`	`if (memcmp(argtypes,clist->args,nargs*sizeof(Oid))==0)`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit52cdbdc

File tree

1 file changed

1 file changed

`‎src/backend/parser/parse_func.c`

0 commit comments