diff options
| author | Vito Caputo <vcaputo@pengaru.com> | 2022-06-09 07:45:16 -0700 |
|---|---|---|
| committer | Vito Caputo <vcaputo@pengaru.com> | 2022-06-10 20:59:03 -0700 |
| commit | 59701ade1169da559dd90c1892e97bb93191e78e (patch) | |
| tree | 37ec90a57bfad9d40220a18ea8476971889f0942 /src/modules | |
| parent | 923aa25cfac8bfcb9e2a8f56398fa8fa7a6c8cc7 (diff) | |
modules/montage: remove vestigial unused variable
This initializer could perform an out-of-bounds read since it
occurs before the n_modules bounds check.
Since the variable isn't even being used anymore just get rid of
this. Also found via ASAN.
Diffstat (limited to 'src/modules')
| -rw-r--r-- | src/modules/montage/montage.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/modules/montage/montage.c b/src/modules/montage/montage.c index 443fcaa..4702564 100644 --- a/src/modules/montage/montage.c +++ b/src/modules/montage/montage.c @@ -192,7 +192,6 @@ static void montage_prepare_frame(til_module_context_t *context, unsigned ticks, static void montage_render_fragment(til_module_context_t *context, unsigned ticks, unsigned cpu, til_fb_fragment_t *fragment) { montage_context_t *ctxt = (montage_context_t *)context; - const til_module_t *module = ctxt->modules[fragment->number]; if (fragment->number >= ctxt->n_modules) { til_fb_fragment_clear(fragment); |