diff options
author | Hans de Goede <[email protected]> | 2023-05-29 11:37:31 +0100 |
---|---|---|
committer | Mauro Carvalho Chehab <[email protected]> | 2023-06-09 15:29:19 +0100 |
commit | e79fae3d15095542db18c4cb2adfbd9e8fda91a4 (patch) | |
tree | a977af32d0141a757eb0eb8547b9e0d75eab8cff /tools/perf/util/trace-event-scripting.c | |
parent | 566f6de6aa283be619e870fae175404b447cdee1 (diff) |
media: atomisp: Remove bogus fh use from atomisp_set_fmt*()
atomisp_set_fmt*() use a local v4l2_subdev_fh declared on the stack,
specifically they use fh.state which is never initialized so when
passing fh.state to atomisp_subdev_set_ffmt() / to
atomisp_subdev_set_selection() these functions are passing random
stack contents as a pointer.
The reason this works is because when the which parameter is
V4L2_SUBDEV_FORMAT_ACTIVE the passed in state is not used.
Remove the bogus fh usage and just pass NULL as state.
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Hans de Goede <[email protected]>
Reviewed-by: Andy Shevchenko <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
Diffstat (limited to 'tools/perf/util/trace-event-scripting.c')
0 files changed, 0 insertions, 0 deletions