simplehash: Free collisions array in SH_STAT

anarazel · anarazel · commit af7e90a27786 · 2024-04-07T19:08:41.000-07:00
While SH_STAT() is only used for debugging, the allocated array can be large, and therefore should be freed. It's unclear why coverity started warning now. Reported-by: Tom Lane <tgl@sss.pgh.pa.us> Reported-by: Coverity Discussion: https://postgr.es/m/3005248.1712538233@sss.pgh.pa.us Backpatch: 12-
diff --git a/src/include/lib/simplehash.h b/src/include/lib/simplehash.h
@@ -1119,6 +1119,9 @@ SH_STAT(SH_TYPE * tb)
 			max_collisions = curcoll;
 	}
 
+	/* large enough to be worth freeing, even if just used for debugging */
+	pfree(collisions);
+
 	if (tb->members > 0)
 	{
 		fillfactor = tb->members / ((double) tb->size);

Original file line number	Diff line number	Diff line change
`@@ -1119,6 +1119,9 @@ SH_STAT(SH_TYPE * tb)`
`1119`	`1119`	`max_collisions = curcoll;`
`1120`	`1120`	`}`
`1121`	`1121`
	`1122`	`+ /* large enough to be worth freeing, even if just used for debugging */`
	`1123`	`+ pfree(collisions);`
	`1124`	`+`
`1122`	`1125`	`if (tb->members > 0)`
`1123`	`1126`	`{`
`1124`	`1127`	`fillfactor = tb->members / ((double) tb->size);`