Commit377790f

committed

Pass sensible value to memset() when randomizing reorderbuffer's tuple slab.

This is entirely harmless, but still wrong. Noticed by coverity.Andres Freund

1 parent329de9f commit377790fCopy full SHA for 377790f

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -456,7 +456,7 @@ ReorderBufferGetTupleBuf(ReorderBuffer *rb)`
`456`	`456`	`tuple=slist_container(ReorderBufferTupleBuf,node,`
`457`	`457`	`slist_pop_head_node(&rb->cached_tuplebufs));`
`458`	`458`	`#ifdefUSE_ASSERT_CHECKING`
`459`		`-memset(tuple,0xdeadbeef,sizeof(ReorderBufferTupleBuf));`
	`459`	`+memset(tuple,0xa9,sizeof(ReorderBufferTupleBuf));`
`460`	`460`	`#endif`
`461`	`461`	`}`
`462`	`462`	`else`

Comments

(0)