diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-01-13 00:28:34 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2012-01-16 08:37:07 -0500 |
commit | 82033bc52abeb266cb3403e783a276d4a3d6cf9f (patch) | |
tree | 56b93ce0cca835d99cacaf8ba998a1f3cb8c36f1 /drivers/media/dvb | |
parent | c24e13713762a30f22468ac2c3d053abacb7672d (diff) |
[media] mb86a20s: fix off by one checks
Clearly ">=" was intended here instead of ">".
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/dvb')
-rw-r--r-- | drivers/media/dvb/frontends/mb86a20s.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/media/dvb/frontends/mb86a20s.c b/drivers/media/dvb/frontends/mb86a20s.c index 7fa3e472cdca..fade566927c3 100644 --- a/drivers/media/dvb/frontends/mb86a20s.c +++ b/drivers/media/dvb/frontends/mb86a20s.c | |||
@@ -402,7 +402,7 @@ static int mb86a20s_get_modulation(struct mb86a20s_state *state, | |||
402 | [2] = 0x8e, /* Layer C */ | 402 | [2] = 0x8e, /* Layer C */ |
403 | }; | 403 | }; |
404 | 404 | ||
405 | if (layer > ARRAY_SIZE(reg)) | 405 | if (layer >= ARRAY_SIZE(reg)) |
406 | return -EINVAL; | 406 | return -EINVAL; |
407 | rc = mb86a20s_writereg(state, 0x6d, reg[layer]); | 407 | rc = mb86a20s_writereg(state, 0x6d, reg[layer]); |
408 | if (rc < 0) | 408 | if (rc < 0) |
@@ -435,7 +435,7 @@ static int mb86a20s_get_fec(struct mb86a20s_state *state, | |||
435 | [2] = 0x8f, /* Layer C */ | 435 | [2] = 0x8f, /* Layer C */ |
436 | }; | 436 | }; |
437 | 437 | ||
438 | if (layer > ARRAY_SIZE(reg)) | 438 | if (layer >= ARRAY_SIZE(reg)) |
439 | return -EINVAL; | 439 | return -EINVAL; |
440 | rc = mb86a20s_writereg(state, 0x6d, reg[layer]); | 440 | rc = mb86a20s_writereg(state, 0x6d, reg[layer]); |
441 | if (rc < 0) | 441 | if (rc < 0) |
@@ -470,7 +470,7 @@ static int mb86a20s_get_interleaving(struct mb86a20s_state *state, | |||
470 | [2] = 0x90, /* Layer C */ | 470 | [2] = 0x90, /* Layer C */ |
471 | }; | 471 | }; |
472 | 472 | ||
473 | if (layer > ARRAY_SIZE(reg)) | 473 | if (layer >= ARRAY_SIZE(reg)) |
474 | return -EINVAL; | 474 | return -EINVAL; |
475 | rc = mb86a20s_writereg(state, 0x6d, reg[layer]); | 475 | rc = mb86a20s_writereg(state, 0x6d, reg[layer]); |
476 | if (rc < 0) | 476 | if (rc < 0) |
@@ -494,7 +494,7 @@ static int mb86a20s_get_segment_count(struct mb86a20s_state *state, | |||
494 | [2] = 0x91, /* Layer C */ | 494 | [2] = 0x91, /* Layer C */ |
495 | }; | 495 | }; |
496 | 496 | ||
497 | if (layer > ARRAY_SIZE(reg)) | 497 | if (layer >= ARRAY_SIZE(reg)) |
498 | return -EINVAL; | 498 | return -EINVAL; |
499 | rc = mb86a20s_writereg(state, 0x6d, reg[layer]); | 499 | rc = mb86a20s_writereg(state, 0x6d, reg[layer]); |
500 | if (rc < 0) | 500 | if (rc < 0) |