On 07/29/2010 07:12 AM, Chris McIntosh wrote:
> +             /*
> +              * @todo Do we need to verify destination pointer
> +              *       "argp" is non-zero?  Is that already done by
> +              *       copy_to_user()?
> +              */

FWIW you need not. (Unless you want argp == NULL being noop and not to
return -EFAULT in that case.)

> +             return copy_to_user(argp,
> +                                 &rar_info,
> +                                 sizeof(rar_info)) ? -EFAULT : 0;

regards,
-- 
js
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to