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
