diff options
| author | Douglas Schilling Landgraf <dougsland@redhat.com> | 2009-08-10 21:07:54 -0400 |
|---|---|---|
| committer | Douglas Schilling Landgraf <dougsland@redhat.com> | 2009-08-10 21:07:54 -0400 |
| commit | b889b807125e28911afda8c7740b18658aeb96fe (patch) | |
| tree | 46fd8163c3d102990281a63a925235dac6408c20 /linux/drivers/media/video/bw-qcam.c | |
| parent | b8961acc4142f124a11a2b33594790dc49dd194a (diff) | |
| download | mediapointer-dvb-s2-b889b807125e28911afda8c7740b18658aeb96fe.tar.gz mediapointer-dvb-s2-b889b807125e28911afda8c7740b18658aeb96fe.tar.bz2 | |
Read buffer overflow
From: Roel Kluin <roel.kluin@gmail.com>
parport[n] is checked before n < MAX_CAMS
Priority: normal
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Douglas Schilling Landgraf <dougsland@redhat.com>
Diffstat (limited to 'linux/drivers/media/video/bw-qcam.c')
| -rw-r--r-- | linux/drivers/media/video/bw-qcam.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/linux/drivers/media/video/bw-qcam.c b/linux/drivers/media/video/bw-qcam.c index 3702baad6..8f6bb845e 100644 --- a/linux/drivers/media/video/bw-qcam.c +++ b/linux/drivers/media/video/bw-qcam.c @@ -993,7 +993,7 @@ static int accept_bwqcam(struct parport *port) if (parport[0] && strncmp(parport[0], "auto", 4) != 0) { /* user gave parport parameters */ - for(n=0; parport[n] && n<MAX_CAMS; n++){ + for (n = 0; n < MAX_CAMS && parport[n]; n++) { char *ep; unsigned long r; r = simple_strtoul(parport[n], &ep, 0); |