mlx5: Return -EFAULT instead of -EPERM
authorDan Carpenter <dan.carpenter@oracle.com>
Wed, 10 Jul 2013 10:58:59 +0000 (13:58 +0300)
committerRoland Dreier <roland@purestorage.com>
Thu, 11 Jul 2013 23:48:45 +0000 (16:48 -0700)
For copy_to/from_user() failure, the correct error code is -EFAULT not
-EPERM.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Or Gerlitz <ogerlitz@mellanox.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
drivers/infiniband/hw/mlx5/mr.c
drivers/net/ethernet/mellanox/mlx5/core/cmd.c
drivers/net/ethernet/mellanox/mlx5/core/debugfs.c

index e2daa8f02476eed8a091c8fe0372fc5b9bb9e7fb..bd41df95b6f0214deb445ab228be8d157373074f 100644 (file)
@@ -171,7 +171,7 @@ static ssize_t size_write(struct file *filp, const char __user *buf,
        int c;
 
        if (copy_from_user(lbuf, buf, sizeof(lbuf)))
-               return -EPERM;
+               return -EFAULT;
 
        c = order2idx(dev, ent->order);
        lbuf[sizeof(lbuf) - 1] = 0;
@@ -208,7 +208,7 @@ static ssize_t size_read(struct file *filp, char __user *buf, size_t count,
                return err;
 
        if (copy_to_user(buf, lbuf, err))
-               return -EPERM;
+               return -EFAULT;
 
        *pos += err;
 
@@ -233,7 +233,7 @@ static ssize_t limit_write(struct file *filp, const char __user *buf,
        int c;
 
        if (copy_from_user(lbuf, buf, sizeof(lbuf)))
-               return -EPERM;
+               return -EFAULT;
 
        c = order2idx(dev, ent->order);
        lbuf[sizeof(lbuf) - 1] = 0;
@@ -270,7 +270,7 @@ static ssize_t limit_read(struct file *filp, char __user *buf, size_t count,
                return err;
 
        if (copy_to_user(buf, lbuf, err))
-               return -EPERM;
+               return -EFAULT;
 
        *pos += err;
 
index c1c0eef896949d66a9f36c093273544d1fadc21a..205753a04cfcb22e8904dbe2e3e203d82df32552 100644 (file)
@@ -693,7 +693,7 @@ static ssize_t dbg_write(struct file *filp, const char __user *buf,
                return -ENOMEM;
 
        if (copy_from_user(lbuf, buf, sizeof(lbuf)))
-               return -EPERM;
+               return -EFAULT;
 
        lbuf[sizeof(lbuf) - 1] = 0;
 
@@ -889,7 +889,7 @@ static ssize_t data_write(struct file *filp, const char __user *buf,
                return -ENOMEM;
 
        if (copy_from_user(ptr, buf, count)) {
-               err = -EPERM;
+               err = -EFAULT;
                goto out;
        }
        dbg->in_msg = ptr;
@@ -919,7 +919,7 @@ static ssize_t data_read(struct file *filp, char __user *buf, size_t count,
 
        copy = min_t(int, count, dbg->outlen);
        if (copy_to_user(buf, dbg->out_msg, copy))
-               return -EPERM;
+               return -EFAULT;
 
        *pos += copy;
 
@@ -949,7 +949,7 @@ static ssize_t outlen_read(struct file *filp, char __user *buf, size_t count,
                return err;
 
        if (copy_to_user(buf, &outlen, err))
-               return -EPERM;
+               return -EFAULT;
 
        *pos += err;
 
@@ -974,7 +974,7 @@ static ssize_t outlen_write(struct file *filp, const char __user *buf,
        dbg->outlen = 0;
 
        if (copy_from_user(outlen_str, buf, count))
-               return -EPERM;
+               return -EFAULT;
 
        outlen_str[7] = 0;
 
index a550a8ef94a192a844c1ea26ab140fcbeca50f4c..4273c06e2e967be08db5bcc99862b723e667eca7 100644 (file)
@@ -148,7 +148,6 @@ static ssize_t average_read(struct file *filp, char __user *buf, size_t count,
        struct mlx5_cmd_stats *stats;
        u64 field = 0;
        int ret;
-       int err;
        char tbuf[22];
 
        if (*pos)
@@ -161,9 +160,8 @@ static ssize_t average_read(struct file *filp, char __user *buf, size_t count,
        spin_unlock(&stats->lock);
        ret = snprintf(tbuf, sizeof(tbuf), "%llu\n", field);
        if (ret > 0) {
-               err = copy_to_user(buf, tbuf, ret);
-               if (err)
-                       return err;
+               if (copy_to_user(buf, tbuf, ret))
+                       return -EFAULT;
        }
 
        *pos += ret;
@@ -418,7 +416,6 @@ static ssize_t dbg_read(struct file *filp, char __user *buf, size_t count,
        char tbuf[18];
        u64 field;
        int ret;
-       int err;
 
        if (*pos)
                return 0;
@@ -445,9 +442,8 @@ static ssize_t dbg_read(struct file *filp, char __user *buf, size_t count,
 
        ret = snprintf(tbuf, sizeof(tbuf), "0x%llx\n", field);
        if (ret > 0) {
-               err = copy_to_user(buf, tbuf, ret);
-               if (err)
-                       return err;
+               if (copy_to_user(buf, tbuf, ret))
+                       return -EFAULT;
        }
 
        *pos += ret;