Remove memory leaks in isolationtester.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 27 Jun 2021 16:45:04 +0000 (12:45 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 27 Jun 2021 16:45:04 +0000 (12:45 -0400)
commit1acab120968a2132e2c40a8ed023e4079867f824
tree13994b13b4bf8d453c7499c1ea3264876008b37e
parentdcb0e243dd3d00af00bb0d15b42c7c883e388b43
Remove memory leaks in isolationtester.

specscanner.l leaked a kilobyte of memory per token of the spec file.
Apparently somebody thought that the introductory code block would be
executed once; but it's once per yylex() call.

A couple of functions in isolationtester.c leaked small amounts of
memory due to not bothering to free one-time allocations.  Might
as well improve these so that valgrind gives this program a clean
bill of health.  Also get rid of an ugly static variable.

Coverity complained about one of the one-time leaks, which led me
to try valgrind'ing isolationtester, which led to discovery of the
larger leak.
src/test/isolation/isolationtester.c
src/test/isolation/specscanner.l