[media] af0933: Don't go past arrays
authorMauro Carvalho Chehab <mchehab@osg.samsung.com>
Tue, 4 Nov 2014 20:35:07 +0000 (17:35 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Wed, 5 Nov 2014 10:09:27 +0000 (08:09 -0200)
X-Patchwork-Delegate: m.chehab@samsung.com
Fixes the following sparse warnings:
drivers/media/dvb-frontends/af9033.c:295 af9033_init() error: buffer overflow 'clock_adc_lut' 11 <= 11
drivers/media/dvb-frontends/af9033.c:300 af9033_init() error: buffer overflow 'clock_adc_lut' 11 <= 11
drivers/media/dvb-frontends/af9033.c:584 af9033_set_frontend() error: buffer overflow 'coeff_lut' 3 <= 3
drivers/media/dvb-frontends/af9033.c:595 af9033_set_frontend() error: buffer overflow 'clock_adc_lut' 11 <= 11

Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Reviewed-by: Antti Palosaari <crope@iki.fi>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/dvb-frontends/af9033.c

index c17e34fd0fb424571a0b6d10d6ce1bfc03a5631b..82ce47bdf5dc79a1e35f6c52f1d9b36b319ac90a 100644 (file)
@@ -291,6 +291,12 @@ static int af9033_init(struct dvb_frontend *fe)
                if (clock_adc_lut[i].clock == dev->cfg.clock)
                        break;
        }
+       if (i == ARRAY_SIZE(clock_adc_lut)) {
+               dev_err(&dev->client->dev,
+                       "Couldn't find ADC config for clock=%d\n",
+                       dev->cfg.clock);
+               goto err;
+       }
 
        adc_cw = af9033_div(dev, clock_adc_lut[i].adc, 1000000ul, 19ul);
        buf[0] = (adc_cw >>  0) & 0xff;
@@ -580,7 +586,15 @@ static int af9033_set_frontend(struct dvb_frontend *fe)
                                break;
                        }
                }
-               ret =  af9033_wr_regs(dev, 0x800001,
+               if (i == ARRAY_SIZE(coeff_lut)) {
+                       dev_err(&dev->client->dev,
+                               "Couldn't find LUT config for clock=%d\n",
+                               dev->cfg.clock);
+                       ret = -EINVAL;
+                       goto err;
+               }
+
+               ret = af9033_wr_regs(dev, 0x800001,
                                coeff_lut[i].val, sizeof(coeff_lut[i].val));
        }
 
@@ -592,6 +606,13 @@ static int af9033_set_frontend(struct dvb_frontend *fe)
                        if (clock_adc_lut[i].clock == dev->cfg.clock)
                                break;
                }
+               if (i == ARRAY_SIZE(clock_adc_lut)) {
+                       dev_err(&dev->client->dev,
+                               "Couldn't find ADC clock for clock=%d\n",
+                               dev->cfg.clock);
+                       ret = -EINVAL;
+                       goto err;
+               }
                adc_freq = clock_adc_lut[i].adc;
 
                /* get used IF frequency */