diff options
author | Simon Arlott <simon@fire.lp0.eu> | 2009-05-12 17:39:28 -0300 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-06-16 18:21:10 -0300 |
commit | eda9f7523b6760bdfc3b9c5001214467b5ecd618 (patch) | |
tree | bedaac0621dfdabfdf9d5bd0c4f0a37590d5e4de /lib | |
parent | 1010ed132727bbf486ac28fd149ccfb0ef5cd2ab (diff) |
V4L/DVB (11841): core: fix potential mutex_unlock without mutex_lock in dvb_dvr_read
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 'lib')
0 files changed, 0 insertions, 0 deletions