NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commitd94d37f

committed

Plug memory leak in index_get_partition

The list of indexes was being leaked when asked for an index thatdoesn't have an index partition in the table partition. Not a commoncase admittedly --and in most cases where it occurs, caller throws anerror anyway-- but worth fixing for cleanliness and in case anythird-party code is calling this function.While at it, remove use of lfirst_oid() to obtain a value we alreadyhave.Author: Justin Pryzby <pryzby@telsasoft.com>Reviewed-by: Michael Paquier <michael@paquier.xyz>Discussion:https://postgr.es/m/20201105203606.GF22691@telsasoft.com

1 parent6a7b55f commitd94d37fCopy full SHA for d94d37f

File tree

1 file changed

-1

lines changed

src/backend/catalog
- partition.c

1 file changed

-1

lines changed

`‎src/backend/catalog/partition.c`

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -170,13 +170,14 @@ index_get_partition(Relation partition, Oid indexId)`
`170`	`170`	`ReleaseSysCache(tup);`
`171`	`171`	`if (!ispartition)`
`172`	`172`	`continue;`
`173`		`-if (get_partition_parent(lfirst_oid(l))==indexId)`
	`173`	`+if (get_partition_parent(partIdx)==indexId)`
`174`	`174`	`{`
`175`	`175`	`list_free(idxlist);`
`176`	`176`	`returnpartIdx;`
`177`	`177`	`}`
`178`	`178`	`}`
`179`	`179`
	`180`	`+list_free(idxlist);`
`180`	`181`	`returnInvalidOid;`
`181`	`182`	`}`
`182`	`183`

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!

Commitd94d37f

File tree

1 file changed

1 file changed

`‎src/backend/catalog/partition.c`

0 commit comments