[patch] [media] s5k6aa: off by one in s5k6aa_enum_frame_interval()

2013-08-23 Thread Dan Carpenter
The check is off by one so we could read one space past the end of the array. Signed-off-by: Dan Carpenter dan.carpen...@oracle.com diff --git a/drivers/media/i2c/s5k6aa.c b/drivers/media/i2c/s5k6aa.c index 789c02a..629a5cd 100644 --- a/drivers/media/i2c/s5k6aa.c +++ b/drivers/media/i2c/s5k6aa.c

Re: [patch] [media] s5k6aa: off by one in s5k6aa_enum_frame_interval()

2013-08-23 Thread Laurent Pinchart
Hi Dan, Thank you for the patch. On Friday 23 August 2013 12:33:06 Dan Carpenter wrote: The check is off by one so we could read one space past the end of the array. Signed-off-by: Dan Carpenter dan.carpen...@oracle.com Acked-by: Laurent Pinchart laurent.pinch...@ideasonboard.com Mauro, I

Re: [patch] [media] s5k6aa: off by one in s5k6aa_enum_frame_interval()

2013-08-23 Thread Sylwester Nawrocki
Hi Laurent, On 08/23/2013 02:54 PM, Laurent Pinchart wrote: Hi Dan, Thank you for the patch. On Friday 23 August 2013 12:33:06 Dan Carpenter wrote: The check is off by one so we could read one space past the end of the array. Signed-off-by: Dan Carpenter dan.carpen...@oracle.com

Re: [patch] [media] s5k6aa: off by one in s5k6aa_enum_frame_interval()

2013-08-23 Thread Laurent Pinchart
Hi Sylwester, On Friday 23 August 2013 15:11:49 Sylwester Nawrocki wrote: On 08/23/2013 02:54 PM, Laurent Pinchart wrote: On Friday 23 August 2013 12:33:06 Dan Carpenter wrote: The check is off by one so we could read one space past the end of the array. Signed-off-by: Dan Carpenter