NotificationsYou must be signed in to change notification settings
Fork5
Star26

Commitae4760d

committed

Fix small query-lifespan memory leak in bulk updates.

When there is an identifiable REPLICA IDENTITY index on the target table,heap_update leaks the id_attrs bitmapset. That's not many bytes, but itadds up over enough rows, since the code typically runs in a query-lifespancontext. Bug introduced in commite55704d, which did a rather poor jobof cloning the existing use-pattern for RelationGetIndexAttrBitmap().Per bug #14293 from Zhou Digoal. Back-patch to 9.4 where the bug wasintroduced.Report: <20160824114320.15676.45171@wrigleys.postgresql.org>

1 parentca9cb94 commitae4760dCopy full SHA for ae4760d

File tree

1 file changed

-0

lines changed

src/backend/access/heap
- heapam.c

1 file changed

-0

lines changed

`‎src/backend/access/heap/heapam.c`

Lines changed: 2 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -3802,6 +3802,7 @@ heap_update(Relation relation, ItemPointer otid, HeapTuple newtup,`
`3802`	`3802`	`ReleaseBuffer(vmbuffer);`
`3803`	`3803`	`bms_free(hot_attrs);`
`3804`	`3804`	`bms_free(key_attrs);`
	`3805`	`+bms_free(id_attrs);`
`3805`	`3806`	`returnresult;`
`3806`	`3807`	`}`
`3807`	`3808`
`@@ -4268,6 +4269,7 @@ heap_update(Relation relation, ItemPointer otid, HeapTuple newtup,`
`4268`	`4269`
`4269`	`4270`	`bms_free(hot_attrs);`
`4270`	`4271`	`bms_free(key_attrs);`
	`4272`	`+bms_free(id_attrs);`
`4271`	`4273`
`4272`	`4274`	`returnHeapTupleMayBeUpdated;`
`4273`	`4275`	`}`

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!

Commitae4760d

File tree

1 file changed

1 file changed

`‎src/backend/access/heap/heapam.c`

0 commit comments