diff options
author | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-05-12 20:39:28 +0000 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-05-12 20:39:28 +0000 |
commit | 3e94e24e59b323cdacd22598b28769753866bfc3 (patch) | |
tree | 171e9b89ed2b5cccd7858b438a6f65b40cfd6849 /v4l2-spec/vidioc-log-status.sgml | |
parent | b474230d34df50ee4b5f01a6264ffb2d41dc5f9e (diff) | |
download | mediapointer-dvb-s2-3e94e24e59b323cdacd22598b28769753866bfc3.tar.gz mediapointer-dvb-s2-3e94e24e59b323cdacd22598b28769753866bfc3.tar.bz2 |
core: fix potential mutex_unlock without mutex_lock in dvb_dvr_read
From: Simon Arlott <simon@fire.lp0.eu>
dvb_dvr_read may unlock the dmxdev mutex and return -ENODEV, except this
function is a file op and will never be called with the mutex held.
There's existing mutex_lock and mutex_unlock around the actual read but
it's commented out. These should probably be uncommented but the read
blocks and this could block another non-blocking reader on the mutex
instead.
This change comments out the extra mutex_unlock.
[akpm@linux-foundation.org: cleanups, simplification]
Signed-off-by: Simon Arlott <simon@fire.lp0.eu>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'v4l2-spec/vidioc-log-status.sgml')
0 files changed, 0 insertions, 0 deletions