media: imon: Fix race getting ictx->lock
authorRicardo Ribalda <ribalda@chromium.org>
Mon, 6 May 2024 21:10:27 +0000 (21:10 +0000)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Fri, 31 May 2024 12:20:40 +0000 (14:20 +0200)
commit24147897507cd3a7d63745d1518a638bf4132238
tree8a76ff4e635b15a110af992806c5e33e69c876e5
parent2052138b7da52ad5ccaf74f736d00f39a1c9198c
media: imon: Fix race getting ictx->lock

Lets fix a race between mutex_is_lock() and mutex_lock().

<-mutex is not locked
if (!mutex_is_locked(&ictx->lock)) {
unlock = true; <- mutex is locked externaly
mutex_lock(&ictx->lock);
}

Let's use mutex_trylock() that does mutex_is_lock() and mutex_lock()
atomically.

Fix the following cocci warning:
drivers/media/rc/imon.c:1167:1-7: preceding lock on line 1153

Fixes: 23ef710e1a6c ("[media] imon: add conditional locking in change_protocol")
Signed-off-by: Ricardo Ribalda <ribalda@chromium.org>
Signed-off-by: Sean Young <sean@mess.org>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
drivers/media/rc/imon.c