Commit1c6bb38

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 parente71c76f commit1c6bb38Copy full SHA for 1c6bb38

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -6528,7 +6528,7 @@ write_item(const void data, Size len, FILE fp)`
`6528`	`6528`	`{`
`6529`	`6529`	`if (fwrite(&len,1,sizeof(len),fp)!=sizeof(len))`
`6530`	`6530`	`elog(FATAL,"could not write init file");`
`6531`		`-if (fwrite(data,1,len,fp)!=len)`
	`6531`	`+if (len>0&&fwrite(data,1,len,fp)!=len)`
`6532`	`6532`	`elog(FATAL,"could not write init file");`
`6533`	`6533`	`}`
`6534`	`6534`

Comments

(0)