lkml.org 
[lkml]   [2018]   [Jul]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 17/32] staging: gasket: annotate ioctl arg with __user
On Thu, Jul 19, 2018 at 07:44:53PM -0700, Todd Poynor wrote:
> >> @@ -549,7 +549,7 @@ struct gasket_driver_desc {
> >> * return -EINVAL. Should return an error status (either -EINVAL or
> >> * the error result of the ioctl being handled).
> >> */
> >> - long (*ioctl_handler_cb)(struct file *filp, uint cmd, ulong arg);
> >> + long (*ioctl_handler_cb)(struct file *filp, uint cmd, void __user *arg);
> >
> > Why are you not using the typedef above?
>
> There's a typedef for the permissions check callback, but not for the
> handler callback. It's a bit confusing, so I tried adding a typedef
> for the handler, but now checkpatch is spanking me for adding new
> typedefs -- maybe I should drop the existing typedef?
>

Someone should update checkpatch to not complain about function typedefs
because those are obviously OK. It wouldn't have even occured to me
that people were avoiding them because of checkpatch...

regards,
dan carpenter

\
 
 \ /
  Last update: 2018-07-30 09:02    [W:0.079 / U:0.376 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site