staging: comedi: comedi.h: remove __user tag from chanlist
authorH Hartley Sweeten <hartleys@visionengravers.com>
Tue, 18 Sep 2012 18:41:04 +0000 (11:41 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 19 Sep 2012 08:36:44 +0000 (09:36 +0100)
The 'chanlist' is passed to the kernel from user space with the
COMEDI_CMD and COMEDI_CMDTEST ioctls. The do_cmd_ioctl() and
do_cmdtest_ioctl() functions in comedi_fops.c copy this data
to/from user space to kernel space correctly.

Unfortunately, this data is copied back into a struct comedi_cmd
so when the cmd is passed down to the drivers they still see the
pointer as __user data. This results is a number of sparse errors
such as:

  warning: dereference of noderef expression

  warning: incorrect type in argument 2 (different address spaces)
     expected void const *<noident>
     got unsigned int [noderef] <asn:1>*chanlist

  warning: incorrect type in argument 3 (different address spaces)
     expected unsigned int *chanlist
     got unsigned int [noderef] <asn:1>*chanlist

  warning: incorrect type in assignment (different address spaces)
     expected unsigned int *ai_chanlist
     got unsigned int [noderef] <asn:1>*chanlist

The two functions in comedi_fops are the only ones that need the
__user tag. Remove the tag so that all the drivers see the chanlist
pointer in the correct address space.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Acked-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/comedi.h

index 76cdb2c8bb7f9258eff9038b00b2fcaeb2f55c28..133f013e0f6d783c0c7dfb5835f9e978edda7902 100644 (file)
@@ -365,7 +365,7 @@ struct comedi_cmd {
        unsigned int stop_src;
        unsigned int stop_arg;
 
-       unsigned int __user *chanlist;  /* channel/range list */
+       unsigned int *chanlist; /* channel/range list */
        unsigned int chanlist_len;
 
        short __user *data; /* data list, size depends on subd flags */