diff options
author | WANG Cong <xiyou.wangcong@gmail.com> | 2017-04-13 21:38:02 +0300 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-04-17 20:28:48 +0300 |
commit | f5001ceab8996d76880004381c188b3b96c8c7af (patch) | |
tree | 2ffa78ba3e97559ff26084d8cf1e90b9ca8b4475 | |
parent | 6b2af241f0766a9d2c01dfa4e5aece433b51ff70 (diff) | |
download | linux-f5001ceab8996d76880004381c188b3b96c8c7af.tar.xz |
kcm: remove a useless copy_from_user()
struct kcm_clone only contains fd, and kcm_clone() only
writes this struct, so there is no need to copy it from user.
Cc: Tom Herbert <tom@herbertland.com>
Signed-off-by: Cong Wang <xiyou.wangcong@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/kcm/kcmsock.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/net/kcm/kcmsock.c b/net/kcm/kcmsock.c index 31762f76cdb5..deca20fb2ce2 100644 --- a/net/kcm/kcmsock.c +++ b/net/kcm/kcmsock.c @@ -1707,11 +1707,7 @@ static int kcm_ioctl(struct socket *sock, unsigned int cmd, unsigned long arg) struct kcm_clone info; struct socket *newsock = NULL; - if (copy_from_user(&info, (void __user *)arg, sizeof(info))) - return -EFAULT; - err = kcm_clone(sock, &info, &newsock); - if (!err) { if (copy_to_user((void __user *)arg, &info, sizeof(info))) { |