diff options
author | Vito Caputo <vcaputo@pengaru.com> | 2022-04-24 21:16:37 -0700 |
---|---|---|
committer | Vito Caputo <vcaputo@pengaru.com> | 2022-04-24 21:16:37 -0700 |
commit | 3427eefac9cf38ac3fa2635c69bb3f0693700cf8 (patch) | |
tree | b078abaf3bb0841b89df3e26319035ff01cd0604 /src/modules/montage/montage.c | |
parent | 2bcc2ca1ed6b7e1fa075770bcb306a3fdff917af (diff) |
*: free setup allocations via til_setup_free()
This should plug a bulk of the setup leaks. Some of the
free_funcs still need to be changed to bespoke ones in modules
that allocate nested things in their respective setup, so those
are still leaking the nested things which are usually just a
small strdup of some kind.
Diffstat (limited to 'src/modules/montage/montage.c')
-rw-r--r-- | src/modules/montage/montage.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/src/modules/montage/montage.c b/src/modules/montage/montage.c index 268d7db..13c8a36 100644 --- a/src/modules/montage/montage.c +++ b/src/modules/montage/montage.c @@ -95,12 +95,7 @@ static void * montage_create_context(unsigned ticks, unsigned num_cpus, til_setu if (module->create_context) /* FIXME errors */ ctxt->contexts[i] = module->create_context(ticks, 1, setup); - /* TODO FIXME: free setup! modules don't currently implement it. - * What should probably happen is the setup should become a til struct - * type having just a free function pointer. Then module setups would - * simply embed this at the start of their private setup struct and return a - * pointer to that as their setup. - */ + til_setup_free(setup); } return ctxt; |