diff options
author | Mauro Carvalho Chehab <mchehab@redhat.com> | 2011-07-15 07:49:06 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2011-07-27 16:56:02 -0400 |
commit | 106ff9e013f60e2f40687d4f8b43c164873b5544 (patch) | |
tree | 0dfa6f4214ff0cefee2484eac911531e93bc040b | |
parent | bded70d296a976da1b52ff5d7f574551fbc503bb (diff) |
Revert "[media] DVB: dvb_frontend: off by one in dtv_property_dump()"
This reverts commit a3e4adf274f86b2363fedaa964297cb38526cef0.
As pointed by Andread Oberritter <obi@linuxtv.org>:
That's wrong, because the array size is DTV_MAX_COMMAND + 1.
Using the ARRAY_SIZE macro instead might reduce the confusion.
Also, changeset 3995223038 already fixed this issue.
Reported-by: Andread Oberritter <obi@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r-- | drivers/media/dvb/dvb-core/dvb_frontend.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/dvb/dvb-core/dvb_frontend.c b/drivers/media/dvb/dvb-core/dvb_frontend.c index 21c9072ed99f..efe9c30605e8 100644 --- a/drivers/media/dvb/dvb-core/dvb_frontend.c +++ b/drivers/media/dvb/dvb-core/dvb_frontend.c | |||
@@ -982,7 +982,7 @@ static void dtv_property_dump(struct dtv_property *tvp) | |||
982 | { | 982 | { |
983 | int i; | 983 | int i; |
984 | 984 | ||
985 | if (tvp->cmd <= 0 || tvp->cmd >= DTV_MAX_COMMAND) { | 985 | if (tvp->cmd <= 0 || tvp->cmd > DTV_MAX_COMMAND) { |
986 | printk(KERN_WARNING "%s: tvp.cmd = 0x%08x undefined\n", | 986 | printk(KERN_WARNING "%s: tvp.cmd = 0x%08x undefined\n", |
987 | __func__, tvp->cmd); | 987 | __func__, tvp->cmd); |
988 | return; | 988 | return; |