diff options
author | Hans de Goede <hdegoede@redhat.com> | 2010-12-30 19:54:33 -0300 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2011-01-19 11:44:54 -0200 |
commit | 27074efa2ee8c1ef07dc5f644104e35d39e43322 (patch) | |
tree | 2ba18e26633ad557b37cc3b42c1cd1381c069d61 /drivers/media/radio | |
parent | 4e770f7602fb2285f3106f98109d0685618ddc22 (diff) | |
download | blackbird-op-linux-27074efa2ee8c1ef07dc5f644104e35d39e43322.tar.gz blackbird-op-linux-27074efa2ee8c1ef07dc5f644104e35d39e43322.zip |
[media] gspca_main: Locking fixes 2
Before this patch vidioc_dqbuf is using its own read_lock, where as
other queue related functions use queue_lock. This means that dqbuf is
accessing several variables in a racy manor. The most important one
being fr_o, which may be changed from underneath dqbuf by vidioc_reqbufs
or vidioc_streamoff. Other variables which it accesses unprotected
are gspca_dev->memory, gspca_dev->streaming and gspca_dev->capt_file.
This patch fixes this by changing vidioc_dqbuf to also use the queue_lock.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Acked-by: Jean-Francois Moine <moinejf@free.fr>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/radio')
0 files changed, 0 insertions, 0 deletions