Fix temporary memory leak in system table index scans
authorPeter Eisentraut <peter@eisentraut.org>
Tue, 3 Dec 2024 08:04:20 +0000 (09:04 +0100)
committerPeter Eisentraut <peter@eisentraut.org>
Tue, 3 Dec 2024 08:04:20 +0000 (09:04 +0100)
commit1acf10549e64c6a52ced570d712fcba1a2f5d1ec
tree27d398dde27f323f29ff6f1963c2d331aede72b9
parent1ba0782ce90cb4261098de59b49ae5cb2326566b
Fix temporary memory leak in system table index scans

Commit 811af9786b introduced palloc() calls into systable_beginscan()
and systable_beginscan_ordered().  But there was no pfree(), as is the
usual style.

It turns out that an ANALYZE of a partitioned table can invoke many
thousand system table index scans, and this memory is not cleaned up
until the end of the command, so this can temporarily leak quite a bit
of memory.  Maybe there are improvements to be made at a higher level
about this, but for now, insert a couple of corresponding pfree()
calls to fix this particular issue.

Reported-by: Justin Pryzby <pryzby@telsasoft.com>
Discussion: https://www.postgresql.org/message-id/Z0XTfIq5xUtbkiIh@pryzbyj2023
src/backend/access/index/genam.c