Pekka Pietikainen wrote:
> +             colors = kmalloc(len,GFP_KERNEL);
> +             if(!colors)
> +                     return -ENOMEM;
> +                     
> +             if (copy_from_user(colors,dc->data,len)) {
> +                     kfree(colors);
> +                     return -EFAULT;
> +             }                       

I thought it's better to avoid kmalloc() here altogether.
Fix comitted to CVS, both 2.6 and 2.4 dvb-kernel.

Thanks,
Johannes


Reply via email to