simplehash: Free collisions array in SH_STAT

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-
REL_13_STABLE
Andres Freund 1 year ago
parent 5fe7bcc676
commit eabf98e949
  1. 3
      src/include/lib/simplehash.h

@ -977,6 +977,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);

Loading…
Cancel
Save