comedi: lift copy_from_user() into callers of __comedi_get_user_cmd()

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>

diff --git a/drivers/staging/comedi/comedi_fops.c b/drivers/staging/comedi/comedi_fops.c
index 10ab24019fa524d767d0950efe98fae50ed91bb0..c136cb2f676a33fc102182a6c98fac848cedbdeb 100644 (file)
--- a/drivers/staging/comedi/comedi_fops.c
+++ b/drivers/staging/comedi/comedi_fops.c
@@ -1649,17 +1649,11 @@ error:
 }
 
 static int __comedi_get_user_cmd(struct comedi_device *dev,
-                                struct comedi_cmd __user *arg,
                                 struct comedi_cmd *cmd)
 {
        struct comedi_subdevice *s;
 
        lockdep_assert_held(&dev->mutex);
-       if (copy_from_user(cmd, arg, sizeof(*cmd))) {
-               dev_dbg(dev->class_dev, "bad cmd address\n");
-               return -EFAULT;
-       }
-
        if (cmd->subdev >= dev->n_subdevices) {
                dev_dbg(dev->class_dev, "%d no such subdevice\n", cmd->subdev);
                return -ENODEV;
@@ -1757,8 +1751,13 @@ static int do_cmd_ioctl(struct comedi_device *dev,
 
        lockdep_assert_held(&dev->mutex);
 
+       if (copy_from_user(&cmd, arg, sizeof(cmd))) {
+               dev_dbg(dev->class_dev, "bad cmd address\n");
+               return -EFAULT;
+       }
+
        /* get the user's cmd and do some simple validation */
-       ret = __comedi_get_user_cmd(dev, arg, &cmd);
+       ret = __comedi_get_user_cmd(dev, &cmd);
        if (ret)
                return ret;
 
@@ -1866,8 +1865,13 @@ static int do_cmdtest_ioctl(struct comedi_device *dev,
 
        lockdep_assert_held(&dev->mutex);
 
+       if (copy_from_user(&cmd, arg, sizeof(cmd))) {
+               dev_dbg(dev->class_dev, "bad cmd address\n");
+               return -EFAULT;
+       }
+
        /* get the user's cmd and do some simple validation */
-       ret = __comedi_get_user_cmd(dev, arg, &cmd);
+       ret = __comedi_get_user_cmd(dev, &cmd);
        if (ret)
                return ret;