diff options
| author | Tom Rix <[email protected]> | 2020-08-10 21:25:18 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <[email protected]> | 2020-08-29 08:16:16 +0200 |
| commit | 780d815dcc9b34d93ae69385a8465c38d423ff0f (patch) | |
| tree | d2144e3d3832c589bbe6d25cee54511b8257a915 /tools/perf/scripts/python/export-to-sqlite.py | |
| parent | fe2b582429b76c3fc73596c58a4631c41c61697d (diff) | |
media: tw5864: check status of tw5864_frameinterval_get
clang static analysis reports this problem
tw5864-video.c:773:32: warning: The left expression of the compound
assignment is an uninitialized value.
The computed value will also be garbage
fintv->stepwise.max.numerator *= std_max_fps;
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
stepwise.max is set with frameinterval, which comes from
ret = tw5864_frameinterval_get(input, &frameinterval);
fintv->stepwise.step = frameinterval;
fintv->stepwise.min = frameinterval;
fintv->stepwise.max = frameinterval;
fintv->stepwise.max.numerator *= std_max_fps;
When tw5864_frameinterval_get() fails, frameinterval is not
set. So check the status and fix another similar problem.
Signed-off-by: Tom Rix <[email protected]>
Signed-off-by: Hans Verkuil <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/export-to-sqlite.py')
0 files changed, 0 insertions, 0 deletions