NotificationsYou must be signed in to change notification settings
Fork28
Star151

Commit47412e0

committed

Fix use of dangling pointer in heap_delete() when logging replica identity

When logging the replica identity of a deleted tuple, XLOG_HEAP_DELETErecords include references of the old tuple. Its data is stored in anintermediate variable used to register this information for the WALrecord, but this variable gets away from the stack when the record getsactually inserted.Spotted by clang's AddressSanitizer.Author: Stas KelvishDiscussion:https://postgr.es/m/085C8825-AD86-4E93-AF80-E26CDF03D1EA@postgrespro.ruBackpatch-through: 9.4

1 parentcf25498 commit47412e0Copy full SHA for 47412e0

File tree

1 file changed

-2

lines changed

src/backend/access/heap
- heapam.c

1 file changed

-2

lines changed

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

Lines changed: 1 addition & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -3343,6 +3343,7 @@ heap_delete(Relation relation, ItemPointer tid,`
`3343`	`3343`	`if (RelationNeedsWAL(relation))`
`3344`	`3344`	`{`
`3345`	`3345`	`xl_heap_deletexlrec;`
	`3346`	`+xl_heap_headerxlhdr;`
`3346`	`3347`	`XLogRecPtrrecptr;`
`3347`	`3348`
`3348`	`3349`	`/* For logical decode we need combocids to properly decode the catalog */`
`@@ -3377,8 +3378,6 @@ heap_delete(Relation relation, ItemPointer tid,`
`3377`	`3378`	`*/`
`3378`	`3379`	`if (old_key_tuple!=NULL)`
`3379`	`3380`	`{`
`3380`		`-xl_heap_headerxlhdr;`
`3381`		`-`
`3382`	`3381`	`xlhdr.t_infomask2=old_key_tuple->t_data->t_infomask2;`
`3383`	`3382`	`xlhdr.t_infomask=old_key_tuple->t_data->t_infomask;`
`3384`	`3383`	`xlhdr.t_hoff=old_key_tuple->t_data->t_hoff;`

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!

Commit47412e0

File tree

1 file changed

1 file changed

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

0 commit comments