NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit2493e2c

committed

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 thememory is allocated in the executor's per-query context which is aboutto get deleted anyway; but it causes severe memory leakage duringcreation or filling of large exclusion-constraint indexes.Also, document that amendscan is supposed to free what ambeginscanallocates. The docs' lack of clarity on that point probably caused thisbug to begin with. (There is discussion of changing that API spec goingforward, 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 commit2a63683(allocating memory during spgrescan instead of spgbeginscan, whichmight be all right if it got cleaned up, but it didn't). And do a bitof code beautification on that commit, too.Discussion:https://postgr.es/m/20181024012314.GA27428@wolff.to

1 parentf107c33 commit2493e2cCopy full SHA for 2493e2c

File tree

2 files changed

+10

-1

lines changed

doc/src/sgml
- indexam.sgml
src/backend/access/spgist
- spgscan.c

2 files changed

+10

-1

lines changed

`‎doc/src/sgml/indexam.sgml`

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -614,7 +614,8 @@ amendscan (IndexScanDesc scan);`
`614`	`614`	`</programlisting>`
`615`	`615`	`End a scan and release resources. The <literal>scan</literal> struct itself`
`616`	`616`	`should not be freed, but any locks or pins taken internally by the`
`617`		`- access method must be released.`
	`617`	`+ access method must be released, as well as any other memory allocated`
	`618`	`+ by <function>ambeginscan</function> and other scan-related functions.`
`618`	`619`	`</para>`
`619`	`620`
`620`	`621`	`<para>`

`‎src/backend/access/spgist/spgscan.c`

Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -240,6 +240,14 @@ spgendscan(IndexScanDesc scan)`
`240`	`240`
`241`	`241`	`MemoryContextDelete(so->tempCxt);`
`242`	`242`	`MemoryContextDelete(so->traversalCxt);`
	`243`	`+`
	`244`	`+if (so->keyData)`
	`245`	`+pfree(so->keyData);`
	`246`	`+`
	`247`	`+if (so->state.deadTupleStorage)`
	`248`	`+pfree(so->state.deadTupleStorage);`
	`249`	`+`
	`250`	`+pfree(so);`
`243`	`251`	`}`
`244`	`252`
`245`	`253`	`/*`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit2493e2c

File tree

2 files changed

2 files changed

`‎doc/src/sgml/indexam.sgml`

`‎src/backend/access/spgist/spgscan.c`

0 commit comments