Fix intra-query memory leak when a SRF returns zero rows.
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 28 Oct 2023 18:04:43 +0000 (14:04 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 28 Oct 2023 18:04:43 +0000 (14:04 -0400)
commit5d7515d7d1168a66a34aed7b9f2c9f4e2660e863
tree7474691e82239470edced4dbba3643c0db22053b
parent877e7b4b9c9938967ee48df17ac6d09462da4214
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