fdo-mirrors/libei
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/libinput/libei.git synced 2026-04-19 13:50:40 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

util: free the test parameters after the run

Browse source 
And also ensure we have the null-termination even if
MUNIT_TEST_MAX_PARAMS are given.

Fixes: 2996a66b37 ("util: add support for parametrized tests")
Part-of: <https://gitlab.freedesktop.org/libinput/libei/-/merge_requests/372>
This commit is contained in:
Peter Hutterer 2025-12-16 10:02:54 +10:00 • committed by Marge Bot
parent d5198d0e53
commit 08d7d5918f
1 changed files with 8 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/util-munit.c
View file

@ -57,7 +57,7 @@ munit_tests_run(int argc, char **argv)
size_t idx = 0;
foreach_test(t) {
size_t nparams = 0;
MunitParameterEnum *parameters = calloc(MUNIT_TEST_MAX_PARAMS, sizeof(*parameters));
MunitParameterEnum *parameters = calloc(MUNIT_TEST_MAX_PARAMS + 1, sizeof(*parameters)); /* null-terminated */
const char *name = NULL;
char **values = NULL;
@ -124,6 +124,13 @@ munit_tests_run(int argc, char **argv)
int rc = munit_suite_main(&suite, setup.userdata, setup.argc, setup.argv);
for (idx = 0; idx < count; idx++) {
MunitParameterEnum *parameters = tests[idx].parameters;
while (parameters && parameters->name) {
free(parameters->name);
strv_free(parameters->values);
parameters++;
}
free(tests[idx].name);
free(tests[idx].parameters);
}