Re: [patch] [media] sh_vou: almost forever loop in sh_vou_try_fmt_vid_out()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Aug 23, 2013 at 3:24 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote:
> The "i < " part of the "i < ARRAY_SIZE()" condition was missing.
>
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

Reviewed-by:  Lad, Prabhakar <prabhakar.lad@xxxxxx>

Regards,
--Prabhakar Lad

>
> diff --git a/drivers/media/platform/sh_vou.c b/drivers/media/platform/sh_vou.c
> index 7a9c5e9..41f612c 100644
> --- a/drivers/media/platform/sh_vou.c
> +++ b/drivers/media/platform/sh_vou.c
> @@ -776,9 +776,10 @@ static int sh_vou_try_fmt_vid_out(struct file *file, void *priv,
>         v4l_bound_align_image(&pix->width, 0, VOU_MAX_IMAGE_WIDTH, 1,
>                               &pix->height, 0, VOU_MAX_IMAGE_HEIGHT, 1, 0);
>
> -       for (i = 0; ARRAY_SIZE(vou_fmt); i++)
> +       for (i = 0; i < ARRAY_SIZE(vou_fmt); i++) {
>                 if (vou_fmt[i].pfmt == pix->pixelformat)
>                         return 0;
> +       }
>
>         pix->pixelformat = vou_fmt[0].pfmt;
>
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux