diff options
| author | Laurent Pinchart <[email protected]> | 2020-04-06 18:39:02 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <[email protected]> | 2020-04-21 13:20:33 +0200 |
| commit | c943b6947f459189419114682ac343fc1e22a954 (patch) | |
| tree | 6154ea3f3095a748ba5bbf5b98efb76fba1c5aae /tools/perf/scripts/python | |
| parent | f0f71ae440a956d51ed1f3e2ba17638c51b7dfa7 (diff) | |
media: imx: utils: Make imx_media_pixfmt handle variable number of codes
The imx_media_pixfmt structures include a codes member that stores
media bus codes as a fixed array of 4 integers. The functions dealing
with the imx_media_pixfmt structures assume that the array of codes is
terminated by a 0 element. This mechanism is fragile, as demonstrated
by several instances of the structure containing 4 non-zero codes.
Fix this by turning the array into a pointer, and providing an
IMX_BUS_FMTS macro to initialize the codes member with a guaranteed 0
element at the end.
[Fixed a NULL deref of the codes pointer in a couple places]
[Added more comments for the struct imx_media_pixfmt members,
including a bold NOTE! for future developers that codes pointer
is NULL for the in-memory-only formats]
Signed-off-by: Laurent Pinchart <[email protected]>
Signed-off-by: Steve Longerbeam <[email protected]>
Signed-off-by: Hans Verkuil <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python')
0 files changed, 0 insertions, 0 deletions