diff options
| author | Hans de Goede <[email protected]> | 2010-12-30 19:54:33 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <[email protected]> | 2011-01-19 11:44:54 -0200 |
| commit | 27074efa2ee8c1ef07dc5f644104e35d39e43322 (patch) | |
| tree | 2ba18e26633ad557b37cc3b42c1cd1381c069d61 /scripts | |
| parent | 4e770f7602fb2285f3106f98109d0685618ddc22 (diff) | |
[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 <[email protected]>
Acked-by: Jean-Francois Moine <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
Diffstat (limited to 'scripts')
0 files changed, 0 insertions, 0 deletions