diff options
| author |   Ksenija Stanojevic <ksenija.stanojevic@gmail.com> | 2015-10-09 15:56:59 +0200 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-10-12 20:35:21 -0700 |
| commit | 4ecc4a18d987e7e9f4b7eff2bf9215ac02c8a980 (patch) | |
| tree | 195e87d5c31211cda26cd45881d7bbde7edddf3d /drivers/staging/rdma/ipath | |
| parent | Staging: most: hdm-i2c: Drop owner assignment from i2c driver (diff) | |
| download | linux-dev-4ecc4a18d987e7e9f4b7eff2bf9215ac02c8a980.tar.xz linux-dev-4ecc4a18d987e7e9f4b7eff2bf9215ac02c8a980.zip | |
Staging: rdma: Use memdup_user
Use memdup_user when user data is immediately copied into the
allocated region.
Issue found using coccinelle.
Signed-off-by: Ksenija Stanojevic<ksenija.stanojevic@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/rdma/ipath')
| -rw-r--r-- | drivers/staging/rdma/ipath/ipath_fs.c | 13 |
1 files changed, 3 insertions, 10 deletions
diff --git a/drivers/staging/rdma/ipath/ipath_fs.c b/drivers/staging/rdma/ipath/ipath_fs.c index 25422a3a7238..796af6867007 100644 --- a/drivers/staging/rdma/ipath/ipath_fs.c +++ b/drivers/staging/rdma/ipath/ipath_fs.c @@ -195,16 +195,9 @@ static ssize_t flash_write(struct file *file, const char __user *buf, goto bail; } - tmp = kmalloc(count, GFP_KERNEL); - if (!tmp) { - ret = -ENOMEM; - goto bail; - } - - if (copy_from_user(tmp, buf, count)) { - ret = -EFAULT; - goto bail_tmp; - } + tmp = memdup_user(buf, count); + if (IS_ERR(tmp)) + return PTR_ERR(tmp); dd = file_inode(file)->i_private; if (ipath_eeprom_write(dd, pos, tmp, count)) { |