diff options
| author | Jose A. Perez de Azpillaga <azpijr@gmail.com> | 2026-03-26 22:34:09 +0100 |
|---|---|---|
| committer | Sakari Ailus <sakari.ailus@linux.intel.com> | 2026-05-20 11:29:32 +0300 |
| commit | f14d52b67e63993e2d184da5f49d884d527dd475 (patch) | |
| tree | df9c3199aaa79a6b287b626021289fb445f7a7f6 /tools/perf/scripts/python/task-analyzer.py | |
| parent | d133bd6ac9beb934875f924e61900990ceb38678 (diff) | |
staging: media: atomisp: improve cleanup robustness in ia_css_stream_destroy_isp2401()
Remove the 'assert(entry)' call. In the atomisp driver, assert() is a
wrapper around BUG(), which intentionally crashes the entire kernel.
This is dangerous and inappropriate for a simple null pointer check
during stream teardown. Replace it with a safe 'if (!entry) continue;'
check.
Change the early 'return' to a 'continue'. In a destruction
path, it is better to proceed with cleaning up as many resources as
possible rather than aborting early, which would result in memory leaks
for the remaining pipes.
Signed-off-by: Jose A. Perez de Azpillaga <azpijr@gmail.com>
Reviewed-by: Dan Carpenter <dan.carpenter@linaro.org>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Diffstat (limited to 'tools/perf/scripts/python/task-analyzer.py')
0 files changed, 0 insertions, 0 deletions
