database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-05-01 01:04:50 +03:00
Code

Remove memory leaks in isolationtester.

Browse Source 
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.
This commit is contained in:
Tom Lane 2021-06-27 12:45:04 -04:00
parent aa2734f5bc
commit bc031cf133
2 changed files with 19 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/test/isolation/isolationtester.c
View File

@ -53,8 +53,8 @@ static int64 max_step_wait = 300 * USECS_PER_SEC;
static void check_testspec(TestSpec *testspec);
static void run_testspec(TestSpec *testspec);
static void run_all_permutations(TestSpec *testspec);
static void run_all_permutations_recurse(TestSpec *testspec, int nsteps,
PermutationStep **steps);
static void run_all_permutations_recurse(TestSpec *testspec, int *piles,
int nsteps, PermutationStep **steps);
static void run_named_permutations(TestSpec *testspec);
static void run_permutation(TestSpec *testspec, int nsteps,
PermutationStep **steps);
@ -361,9 +361,9 @@ check_testspec(TestSpec *testspec)
fprintf(stderr, "unused step name: %s\n", allsteps[i]->name);
}
}
}
static int *piles;
free(allsteps);
}
/*
* Run the permutations specified in the spec, or all if none were
@ -388,6 +388,7 @@ run_all_permutations(TestSpec *testspec)
int i;
PermutationStep *steps;
PermutationStep **stepptrs;
int *piles;
/* Count the total number of steps in all sessions */
nsteps = 0;
@ -413,11 +414,16 @@ run_all_permutations(TestSpec *testspec)
for (i = 0; i < testspec->nsessions; i++)
piles[i] = 0;
run_all_permutations_recurse(testspec, 0, stepptrs);
run_all_permutations_recurse(testspec, piles, 0, stepptrs);
free(steps);
free(stepptrs);
free(piles);
}
static void
run_all_permutations_recurse(TestSpec *testspec, int nsteps, PermutationStep **steps)
run_all_permutations_recurse(TestSpec *testspec, int *piles,
int nsteps, PermutationStep **steps)
{
int i;
bool found = false;
@ -439,7 +445,7 @@ run_all_permutations_recurse(TestSpec *testspec, int nsteps, PermutationStep **s
piles[i]++;
run_all_permutations_recurse(testspec, nsteps + 1, steps);
run_all_permutations_recurse(testspec, piles, nsteps + 1, steps);
piles[i]--;

8
src/test/isolation/specscanner.l
View File

@ -52,8 +52,12 @@ self [,()*]
%%
%{
litbuf = pg_malloc(LITBUF_INIT);
litbufsize = LITBUF_INIT;
/* Allocate litbuf in first call of yylex() */
if (litbuf == NULL)
{
litbuf = pg_malloc(LITBUF_INIT);
litbufsize = LITBUF_INIT;
}
%}
/* Keywords (must appear before the {identifier} rule!) */