Commit8014c61

committed

Don't call fwrite() with len == 0 when writing out relcache init file.

Noticed via -fsanitize=undefined.Backpatch to all branches, for the same reasons as46ab07f.Discussion:https://postgr.es/m/20220323173537.ll7klrglnp4gn2um@alap3.anarazel.deBackpatch: 10-

1 parent7c163aa commit8014c61Copy full SHA for 8014c61

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -6365,7 +6365,7 @@ write_item(const void data, Size len, FILE fp)`
`6365`	`6365`	`{`
`6366`	`6366`	`if (fwrite(&len,1,sizeof(len),fp)!=sizeof(len))`
`6367`	`6367`	`elog(FATAL,"could not write init file");`
`6368`		`-if (fwrite(data,1,len,fp)!=len)`
	`6368`	`+if (len>0&&fwrite(data,1,len,fp)!=len)`
`6369`	`6369`	`elog(FATAL,"could not write init file");`
`6370`	`6370`	`}`
`6371`	`6371`

Comments

(0)