diff options
| author |   Olaf Hering <olaf@aepfle.de> | 2015-12-14 16:01:37 -0800 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-12-14 19:12:21 -0800 |
| commit | b00359642c2427da89dc8f77daa2c9e8a84e6d76 (patch) | |
| tree | b028b339fb4e2b4e02bb9ce1519b53e2f84950ae /drivers/hv | |
| parent | Drivers: hv: util: catch allocation errors (diff) | |
| download | linux-dev-b00359642c2427da89dc8f77daa2c9e8a84e6d76.tar.xz linux-dev-b00359642c2427da89dc8f77daa2c9e8a84e6d76.zip | |
Drivers: hv: utils: use memdup_user in hvt_op_write
Use memdup_user to handle OOM.
Fixes: 14b50f80c32d ('Drivers: hv: util: introduce hv_utils_transport abstraction')
Signed-off-by: Olaf Hering <olaf@aepfle.de>
Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/hv')
| -rw-r--r-- | drivers/hv/hv_utils_transport.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/hv/hv_utils_transport.c b/drivers/hv/hv_utils_transport.c index 1505ee6e6605..24b2766a6d34 100644 --- a/drivers/hv/hv_utils_transport.c +++ b/drivers/hv/hv_utils_transport.c @@ -80,11 +80,10 @@ static ssize_t hvt_op_write(struct file *file, const char __user *buf, hvt = container_of(file->f_op, struct hvutil_transport, fops); - inmsg = kzalloc(count, GFP_KERNEL); - if (copy_from_user(inmsg, buf, count)) { - kfree(inmsg); - return -EFAULT; - } + inmsg = memdup_user(buf, count); + if (IS_ERR(inmsg)) + return PTR_ERR(inmsg); + if (hvt->on_msg(inmsg, count)) return -EFAULT; kfree(inmsg); |