diff options
| author | Mauro Carvalho Chehab | 2013-04-08 02:06:12 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab | 2013-04-08 12:11:06 +0200 |
| commit | b1f89331190820c42397156063fe5e39bc514e3b (patch) | |
| tree | f56386e354aa7c2f4a563bfad8fc6f4b0e99a770 /drivers/media/dvb-frontends/mb86a20s.c | |
| parent | [media] mb86a20s: Use 'layer' instead of 'i' on all places (diff) | |
| download | kernel-qcow2-linux-b1f89331190820c42397156063fe5e39bc514e3b.tar.gz kernel-qcow2-linux-b1f89331190820c42397156063fe5e39bc514e3b.tar.xz kernel-qcow2-linux-b1f89331190820c42397156063fe5e39bc514e3b.zip | |
[media] mb86a20s: Fix estimate_rate setting
As reported by Dan Carpenter <dan.carpenter@oracle.com>:
Smatch warnings:
drivers/media/dvb-frontends/mb86a20s.c:644 mb86a20s_layer_bitrate() error: buffer overflow 'state->estimated_rate' 3 <= 3
What happens there is that estimate_rate index should be the layer
number, and not the guard interval.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/dvb-frontends/mb86a20s.c')
| -rw-r--r-- | drivers/media/dvb-frontends/mb86a20s.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/dvb-frontends/mb86a20s.c b/drivers/media/dvb-frontends/mb86a20s.c index f2565896ce42..d25df75ed404 100644 --- a/drivers/media/dvb-frontends/mb86a20s.c +++ b/drivers/media/dvb-frontends/mb86a20s.c @@ -642,7 +642,7 @@ static void mb86a20s_layer_bitrate(struct dvb_frontend *fe, u32 layer, __func__, 'A' + layer, segment * isdbt_rate[m][f][i]/1000, rate, rate); - state->estimated_rate[i] = rate; + state->estimated_rate[layer] = rate; } |