diff options
| author | Alexander Korotkov <akorotkov@postgresql.org> | 2022-02-14 03:26:55 +0300 |
|---|---|---|
| committer | Alexander Korotkov <akorotkov@postgresql.org> | 2022-02-14 03:32:31 +0300 |
| commit | c76665edce3d4db4e509723e91742569e9f46817 (patch) | |
| tree | d1ba24e85bb91c1fd14a4ffece53a6c7838aa729 /src/backend/executor/nodeIndexscan.c | |
| parent | ae27b1acc43d58badad356a4da8fc0c72d46d7fb (diff) | |
| download | postgresql-c76665edce3d4db4e509723e91742569e9f46817.tar.gz postgresql-c76665edce3d4db4e509723e91742569e9f46817.zip | |
Fix memory leak in IndexScan node with reordering
Fix ExecReScanIndexScan() to free the referenced tuples while emptying the
priority queue. Backpatch to all supported versions.
Discussion: https://postgr.es/m/CAHqSB9gECMENBQmpbv5rvmT3HTaORmMK3Ukg73DsX5H7EJV7jw%40mail.gmail.com
Author: Aliaksandr Kalenik
Reviewed-by: Tom Lane, Alexander Korotkov
Backpatch-through: 10
Diffstat (limited to 'src/backend/executor/nodeIndexscan.c')
| -rw-r--r-- | src/backend/executor/nodeIndexscan.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/backend/executor/nodeIndexscan.c b/src/backend/executor/nodeIndexscan.c index 2fffb1b4371..add29b3733b 100644 --- a/src/backend/executor/nodeIndexscan.c +++ b/src/backend/executor/nodeIndexscan.c @@ -574,8 +574,12 @@ ExecReScanIndexScan(IndexScanState *node) /* flush the reorder queue */ if (node->iss_ReorderQueue) { + HeapTuple tuple; while (!pairingheap_is_empty(node->iss_ReorderQueue)) - reorderqueue_pop(node); + { + tuple = reorderqueue_pop(node); + heap_freetuple(tuple); + } } /* reset index scan */ |