Fix wrong allocation size in c8f621c.

anarazel · anarazel · commit fd45d16f6212 · 2016-03-06T16:27:20.000-08:00
In c8f621c I forgot to account for MAXALIGN when allocating a new tuplebuf in ReorderBufferGetTupleBuf(). That happens to currently not cause active problems on a number of platforms because the affected pointer is already aligned, but others, like ppc and hppa, trigger this in the regression test, due to a debug memset clearing memory. Fix that. Backpatch: 9.4, like the previous commit.
diff --git a/src/backend/replication/logical/reorderbuffer.c b/src/backend/replication/logical/reorderbuffer.c
@@ -483,7 +483,8 @@ ReorderBufferGetTupleBuf(ReorderBuffer *rb, Size tuple_len)
 	{
 		tuple = (ReorderBufferTupleBuf *)
 			MemoryContextAlloc(rb->context,
-							   sizeof(ReorderBufferTupleBuf) + alloc_len);
+							   sizeof(ReorderBufferTupleBuf) +
+							   MAXIMUM_ALIGNOF + alloc_len);
 		tuple->alloc_tuple_size = alloc_len;
 		tuple->tuple.t_data = ReorderBufferTupleBufData(tuple);
 	}

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