diff options
| author | Hans de Goede <[email protected]> | 2023-07-30 17:33:43 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <[email protected]> | 2023-08-10 07:58:33 +0200 |
| commit | 9e2a90d75662d2cad22d8e3f3358c1b5392f037d (patch) | |
| tree | 346bf02a00cf8a7f7c52c1eead67f06261d33db9 /tools/perf/scripts/python/export-to-sqlite.py | |
| parent | 7b4846b6515483396af706329cb51794eb6afb6d (diff) | |
media: atomisp: Fix me->stages error checking in sh_css_sp_init_pipeline()
The current error-checking of me->stages in sh_css_sp_init_pipeline()
has some issues / weirdness:
1. It is checked at the top of the function, but only using the atomisp
custom assert() macro which e.g. smatch does not recognize
2. It is first dereferenced in "first_binary = me->stages->binary", but
outside of the assert it is checked much later, triggering the following
smatch warning:
drivers/staging/media/atomisp/pci/sh_css_sp.c:1255 sh_css_sp_init_pipeline()
warn: variable dereferenced before check 'me->stages' (see line 1224)
Drop the custom assert() calls (note 'me' is never NULL) and instead add
a regular check for me->stages not being set.
Reported-by: Hans Verkuil <[email protected]>
Closes: https://lore.kernel.org/linux-media/[email protected]/
Signed-off-by: Hans de Goede <[email protected]>
Signed-off-by: Hans Verkuil <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/export-to-sqlite.py')
0 files changed, 0 insertions, 0 deletions