From mboxrd@z Thu Jan 1 00:00:00 1970 From: Antti Palosaari Date: Wed, 24 Sep 2014 10:48:47 +0000 Subject: Re: [patch] [media] hackrf: harmless off by one in debug code Message-Id: <5422A18F.9090106@iki.fi> List-Id: References: <20140924103639.GB15107@mwanda> In-Reply-To: <20140924103639.GB15107@mwanda> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Dan Carpenter Cc: Mauro Carvalho Chehab , linux-media@vger.kernel.org, kernel-janitors@vger.kernel.org Acked-by: Antti Palosaari Reviewed-by: Antti Palosaari Antti On 09/24/2014 01:36 PM, Dan Carpenter wrote: > My static checker complains that "i" could be one element beyond the end > of the array. > > Signed-off-by: Dan Carpenter > > diff --git a/drivers/media/usb/hackrf/hackrf.c b/drivers/media/usb/hackrf/hackrf.c > index 328b5ba..fd1fa41 100644 > --- a/drivers/media/usb/hackrf/hackrf.c > +++ b/drivers/media/usb/hackrf/hackrf.c > @@ -932,7 +932,7 @@ static int hackrf_set_bandwidth(struct hackrf_dev *dev) > dev->bandwidth->val = bandwidth; > dev->bandwidth->cur.val = bandwidth; > > - dev_dbg(dev->dev, "bandwidth selected=%d\n", bandwidth_lut[i].freq); > + dev_dbg(dev->dev, "bandwidth selected=%d\n", bandwidth); > > u16tmp = 0; > u16tmp |= ((bandwidth >> 0) & 0xff) << 0; > -- http://palosaari.fi/