summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVito Caputo <vcaputo@pengaru.com>2022-06-09 07:45:16 -0700
committerVito Caputo <vcaputo@pengaru.com>2022-06-10 20:59:03 -0700
commit59701ade1169da559dd90c1892e97bb93191e78e (patch)
tree37ec90a57bfad9d40220a18ea8476971889f0942
parent923aa25cfac8bfcb9e2a8f56398fa8fa7a6c8cc7 (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.
-rw-r--r--src/modules/montage/montage.c1
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);
© All Rights Reserved