Fix intra-query memory leak when a SRF returns zero rows.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 28 Oct 2023 18:04:42 +0000 (14:04 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 28 Oct 2023 18:05:01 +0000 (14:05 -0400)
commit237f8765dfd9149471d37f3754d15cef888338a8
tree6b9f988a6a704757ee4d49e95d009b10cc5ecd29
parent4d42b4edc6def25cab63894a2d351d597606492d
Fix intra-query memory leak when a SRF returns zero rows.

When looping around after finding that the set-returning function
returned zero rows for the current input tuple, ExecProjectSet
neglected to reset either of the two memory contexts it's
responsible for cleaning out.  Typically this wouldn't cause much
problem, because once the SRF does return at least one row, the
contexts would get reset on the next call.  However, if the SRF
returns no rows for many input tuples in succession, quite a lot
of memory could be transiently consumed.

To fix, make sure we reset both contexts while looping around.

Per bug #18172 from Sergei Kornilov.  Back-patch to all supported
branches.

Discussion: https://postgr.es/m/18172-9b8c5fc1d676ded3@postgresql.org
src/backend/executor/nodeProjectSet.c