]> granicus.if.org Git - postgresql/commit
Remove no-longer-helpful reliance on fixed-size local array.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 21 Jul 2019 15:42:11 +0000 (11:42 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 21 Jul 2019 15:42:11 +0000 (11:42 -0400)
commit330cafdfaa11ebe53e3e59688acac1577ae0cb34
tree726d8cc22596d03d317be3d1f77ed2e61781a109
parent90317ab7e64bd2d855c73a6ba579de6d04a7b25c
Remove no-longer-helpful reliance on fixed-size local array.

Coverity complained about this code, apparently because it uses a local
array of size FUNC_MAX_ARGS without a guard that the input argument list
is no longer than that.  (Not sure why it complained today, since this
code's been the same for a long time; possibly it re-analyzed everything
the List API change touched?)

Rather than add a guard, though, let's just get rid of the local array
altogether.  It was only there to avoid list_nth() calls, and those are
no longer expensive.
src/backend/parser/parse_func.c