projects
/
openwrt
/
staging
/
blogic.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
ed18d0c
)
V4L/DVB (12338): cx18: Read buffer overflow
author
Andy Walls
<
[email protected]
>
Fri, 24 Jul 2009 00:51:29 +0000
(21:51 -0300)
committer
Mauro Carvalho Chehab
<
[email protected]
>
Thu, 13 Aug 2009 23:39:02 +0000
(20:39 -0300)
This mistakenly tested against sizeof(freqs) instead of the array size. Due to
the mask the only illegal value possible was 3.
Reported-by: Roel Kluin <
[email protected]
>
Signed-off-by: Andy Walls <
[email protected]
>
Signed-off-by: Mauro Carvalho Chehab <
[email protected]
>
drivers/media/video/cx18/cx18-controls.c
patch
|
blob
|
history
diff --git
a/drivers/media/video/cx18/cx18-controls.c
b/drivers/media/video/cx18/cx18-controls.c
index 5136df198338bbbe6855889b84cf0b5486252b2d..93f0dae01350316d9c13110504df129baa4f5ecb 100644
(file)
--- a/
drivers/media/video/cx18/cx18-controls.c
+++ b/
drivers/media/video/cx18/cx18-controls.c
@@
-20,6
+20,7
@@
* Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA
* 02111-1307 USA
*/
+#include <linux/kernel.h>
#include "cx18-driver.h"
#include "cx18-cards.h"
@@
-317,7
+318,7
@@
int cx18_s_ext_ctrls(struct file *file, void *fh, struct v4l2_ext_controls *c)
idx = p.audio_properties & 0x03;
/* The audio clock of the digitizer must match the codec sample
rate otherwise you get some very strange effects. */
- if (idx <
sizeof
(freqs))
+ if (idx <
ARRAY_SIZE
(freqs))
cx18_call_all(cx, audio, s_clock_freq, freqs[idx]);
return err;
}