]> git.baikalelectronics.ru Git - kernel.git/commit
media: smiapp: fix timeout checking in smiapp_read_nvm
authorColin Ian King <colin.king@canonical.com>
Wed, 25 Apr 2018 15:04:21 +0000 (11:04 -0400)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Thu, 17 May 2018 10:22:08 +0000 (06:22 -0400)
commit9aa2f92dadc9c295aacb0a540961de4464701924
tree06b81a6d685bdab447c78dd27ae73daf127c331b
parent754482ac34ad7af36a0bea48dbbf66758e7ada8c
media: smiapp: fix timeout checking in smiapp_read_nvm

The current code decrements the timeout counter i and the end of
each loop i is incremented, so the check for timeout will always
be false and hence the timeout mechanism is just a dead code path.
Potentially, if the RD_READY bit is not set, we could end up in
an infinite loop.

Fix this so the timeout starts from 1000 and decrements to zero,
if at the end of the loop i is zero we have a timeout condition.

Detected by CoverityScan, CID#1324008 ("Logically dead code")

Fixes: 0a0611f78092 ("[media] smiapp: Add driver")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/media/i2c/smiapp/smiapp-core.c