aboutsummaryrefslogtreecommitdiff
path: root/doc/src
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2018-10-31 17:04:42 -0400
committerTom Lane <tgl@sss.pgh.pa.us>2018-10-31 17:05:03 -0400
commit696b0c5fd0a8765fe6dfd075a30be06b448fd615 (patch)
tree2c7d3b075ed6ef1a3c3bb6e5520587c4ba6ac561 /doc/src
parentc4ab62f9acc0389697042cb642aa74beabdc8cc7 (diff)
downloadpostgresql-696b0c5fd0a8765fe6dfd075a30be06b448fd615.tar.gz
postgresql-696b0c5fd0a8765fe6dfd075a30be06b448fd615.zip
Fix memory leak in repeated SPGIST index scans.
spgendscan neglected to pfree all the memory allocated by spgbeginscan. It's possible to get away with that in most normal queries, since the memory is allocated in the executor's per-query context which is about to get deleted anyway; but it causes severe memory leakage during creation or filling of large exclusion-constraint indexes. Also, document that amendscan is supposed to free what ambeginscan allocates. The docs' lack of clarity on that point probably caused this bug to begin with. (There is discussion of changing that API spec going forward, but I don't think it'd be appropriate for the back branches.) Per report from Bruno Wolff. It's been like this since the beginning, so back-patch to all active branches. In HEAD, also fix an independent leak caused by commit 2a6368343 (allocating memory during spgrescan instead of spgbeginscan, which might be all right if it got cleaned up, but it didn't). And do a bit of code beautification on that commit, too. Discussion: https://postgr.es/m/20181024012314.GA27428@wolff.to
Diffstat (limited to 'doc/src')
-rw-r--r--doc/src/sgml/indexam.sgml3
1 files changed, 2 insertions, 1 deletions
diff --git a/doc/src/sgml/indexam.sgml b/doc/src/sgml/indexam.sgml
index beb99d1831a..d758a4987dc 100644
--- a/doc/src/sgml/indexam.sgml
+++ b/doc/src/sgml/indexam.sgml
@@ -614,7 +614,8 @@ amendscan (IndexScanDesc scan);
</programlisting>
End a scan and release resources. The <literal>scan</literal> struct itself
should not be freed, but any locks or pins taken internally by the
- access method must be released.
+ access method must be released, as well as any other memory allocated
+ by <function>ambeginscan</function> and other scan-related functions.
</para>
<para>