NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitfb3ea04

committed

Fix wrong allocation size inc8f621c.

Inc8f621c I forgot to account for MAXALIGN when allocating a newtuplebuf in ReorderBufferGetTupleBuf(). That happens to currently notcause active problems on a number of platforms because the affectedpointer is already aligned, but others, like ppc and hppa, trigger thisin the regression test, due to a debug memset clearing memory.Fix that.Backpatch: 9.4, like the previous commit.

1 parent3fa4715 commitfb3ea04Copy full SHA for fb3ea04

File tree

1 file changed

-1

lines changed

src/backend/replication/logical
- reorderbuffer.c

1 file changed

-1

lines changed

`‎src/backend/replication/logical/reorderbuffer.c`

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -481,7 +481,8 @@ ReorderBufferGetTupleBuf(ReorderBuffer *rb, Size tuple_len)`
`481`	`481`	`{`
`482`	`482`	`tuple= (ReorderBufferTupleBuf*)`
`483`	`483`	`MemoryContextAlloc(rb->context,`
`484`		`-sizeof(ReorderBufferTupleBuf)+alloc_len);`
	`484`	`+sizeof(ReorderBufferTupleBuf)+`
	`485`	`+MAXIMUM_ALIGNOF+alloc_len);`
`485`	`486`	`tuple->alloc_tuple_size=alloc_len;`
`486`	`487`	`tuple->tuple.t_data=ReorderBufferTupleBufData(tuple);`
`487`	`488`	`}`

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!

Commitfb3ea04

File tree

1 file changed

1 file changed

`‎src/backend/replication/logical/reorderbuffer.c`

0 commit comments