gistendscan() forgot to free so->giststate.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 16 Sep 2011 08:27:49 +0000 (04:27 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 16 Sep 2011 08:27:49 +0000 (04:27 -0400)
commit0a6cc28500b7a8db7a27cbd0d75e18837fb2e367
tree128d6c3e8afa7b5eac4fa66a115f4c9a61b28ef9
parente6faf910d75027bdce7cd0f2033db4e912592bcc
gistendscan() forgot to free so->giststate.

This oversight led to a massive memory leak --- upwards of 10KB per tuple
--- during creation-time verification of an exclusion constraint based on a
GIST index.  In most other scenarios it'd just be a leak of 10KB that would
be recovered at end of query, so not too significant; though perhaps the
leak would be noticeable in a situation where a GIST index was being used
in a nestloop inner indexscan.  In any case, it's a real leak of long
standing, so patch all supported branches.  Per report from Harald Fuchs.
src/backend/access/gist/gistscan.c