diff options
author | Geliang Tang <geliangtang@gmail.com> | 2017-05-06 11:42:22 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-05-08 15:02:10 -0400 |
commit | 871ff2ebe0af0d7f1050bfc8bce62fddea2b12ec (patch) | |
tree | 2a1ad5d19e8c0902c8d265d4ff05afce75fe8a50 | |
parent | 294316a4af16cd5c355a80bd2dc7ea803b88f6ba (diff) |
yam: use memdup_user
Use memdup_user() helper instead of open-coding to simplify the code.
Signed-off-by: Geliang Tang <geliangtang@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/hamradio/yam.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/drivers/net/hamradio/yam.c b/drivers/net/hamradio/yam.c index b6891ada1d7b..7a7c5224a336 100644 --- a/drivers/net/hamradio/yam.c +++ b/drivers/net/hamradio/yam.c | |||
@@ -976,12 +976,10 @@ static int yam_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd) | |||
976 | case SIOCYAMSMCS: | 976 | case SIOCYAMSMCS: |
977 | if (netif_running(dev)) | 977 | if (netif_running(dev)) |
978 | return -EINVAL; /* Cannot change this parameter when up */ | 978 | return -EINVAL; /* Cannot change this parameter when up */ |
979 | if ((ym = kmalloc(sizeof(struct yamdrv_ioctl_mcs), GFP_KERNEL)) == NULL) | 979 | ym = memdup_user(ifr->ifr_data, |
980 | return -ENOBUFS; | 980 | sizeof(struct yamdrv_ioctl_mcs)); |
981 | if (copy_from_user(ym, ifr->ifr_data, sizeof(struct yamdrv_ioctl_mcs))) { | 981 | if (IS_ERR(ym)) |
982 | kfree(ym); | 982 | return PTR_ERR(ym); |
983 | return -EFAULT; | ||
984 | } | ||
985 | if (ym->bitrate > YAM_MAXBITRATE) { | 983 | if (ym->bitrate > YAM_MAXBITRATE) { |
986 | kfree(ym); | 984 | kfree(ym); |
987 | return -EINVAL; | 985 | return -EINVAL; |