Commit 106ff9e0 authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

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: default avatarAndread Oberritter <obi@linuxtv.org>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent bded70d2
...@@ -982,7 +982,7 @@ static void dtv_property_dump(struct dtv_property *tvp) ...@@ -982,7 +982,7 @@ static void dtv_property_dump(struct dtv_property *tvp)
{ {
int i; int i;
if (tvp->cmd <= 0 || tvp->cmd >= DTV_MAX_COMMAND) { if (tvp->cmd <= 0 || tvp->cmd > DTV_MAX_COMMAND) {
printk(KERN_WARNING "%s: tvp.cmd = 0x%08x undefined\n", printk(KERN_WARNING "%s: tvp.cmd = 0x%08x undefined\n",
__func__, tvp->cmd); __func__, tvp->cmd);
return; return;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment