]> granicus.if.org Git - postgresql/commit
Fix failure to account for memory used by tuplestore_putvalues().
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 15 Jun 2011 18:05:22 +0000 (14:05 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 15 Jun 2011 18:05:39 +0000 (14:05 -0400)
commit3cfecae3a34853c631eeb75671c6b82638e8e0aa
tree0c56414fd5ea2f5de7bdb2489868b9e9596dee34
parentdc014e0446f5bfb52129fc6781a98b236d8166c6
Fix failure to account for memory used by tuplestore_putvalues().

This oversight could result in a tuplestore using much more than the
intended amount of memory.  It would only happen in a code path that loaded
a tuplestore via tuplestore_putvalues(), and many of those won't emit huge
amounts of data; but cases such as holdable cursors and plpgsql's RETURN
NEXT command could have the problem.  The fix ensures that the tuplestore
will switch to write-to-disk mode when it overruns work_mem.

The potential overrun was finite, because we would still count the space
used by the tuple pointer array, so the tuplestore code would eventually
flip into write-to-disk mode anyway.  When storing wide tuples we would
go far past the expected work_mem usage before that happened; but this
may account for the lack of prior reports.

Back-patch to 8.4, where tuplestore_putvalues was introduced.

Per bug #6061 from Yann Delorme.
src/backend/utils/sort/tuplestore.c