]> granicus.if.org Git - postgresql/commit
Repair double-free in SP-GIST rescan (bug #15378)
authorAndrew Gierth <rhodiumtoad@postgresql.org>
Tue, 11 Sep 2018 17:14:19 +0000 (18:14 +0100)
committerAndrew Gierth <rhodiumtoad@postgresql.org>
Tue, 11 Sep 2018 18:19:55 +0000 (19:19 +0100)
commit84a3a1e55c86a104a8e259a43ed31814e010954f
treebae894b09963a7725a947aa40886440e9d7e0be2
parentf9e66f2fbbb49a493045c8d8086a9b15d95b8f18
Repair double-free in SP-GIST rescan (bug #15378)

spgrescan would first reset traversalCxt, and then traverse a
potentially non-empty stack containing pointers to traversalValues
which had been allocated in those contexts, freeing them a second
time. This bug originates in commit ccd6eb49a where traversalValue was
introduced.

Repair by traversing the stack before the context reset; this isn't
ideal, since it means doing retail pfree in a context that's about to
be reset, but the freeing of a stack entry is also done in other
places in the code during the scan so it's not worth trying to
refactor it further. Regression test added.

Backpatch to 9.6 where the problem was introduced.

Per bug #15378; analysis and patch by me, originally from a report on
IRC by user velix; see also PostGIS ticket #4174; review by Alexander
Korotkov.

Discussion: https://postgr.es/m/153663176628.23136.11901365223750051490@wrigleys.postgresql.org
src/backend/access/spgist/spgscan.c
src/test/regress/expected/spgist.out
src/test/regress/sql/spgist.sql