On Wed, 2012-05-09 at 22:47 +0800, Hein Tibosch wrote:
> Tomi,
> 
> 
> In drivers/video/omap2/omapfb/omapfb-main.c:
> 
>  static int omapfb_parse_vram_param(const char *param, int max_entries,
>               unsigned long *sizes, unsigned long *paddrs)
>  {
>       int fbnum;
>       unsigned long size;
>       unsigned long paddr = 0;
>       char *p, *start;
>       start = (char *)param;
>       while (1) {
>               p = start;
>               fbnum = simple_strtoul(p, &p, 10);
> -             if (p == param)
> +             if (p == start)
> 
> correct?

Yes, looks like a correct fix. I'll cook up a patch.

How did you encounter the bug? What was the outcome?

 Tomi

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to