diff options
| author | Andre Guedes <andre.guedes@openbossa.org> | 2014-03-24 16:08:48 -0300 |
|---|---|---|
| committer | Marcel Holtmann <marcel@holtmann.org> | 2014-03-26 09:31:34 -0700 |
| commit | 4408dd15d9cfcf78b819d2d4a5b43dc36056cab8 (patch) | |
| tree | 6a2f1035ceec0615cb8688b120aa58812e229853 | |
| parent | 4af605d8c4d3cf5170fdb40b5c77ea133761d2df (diff) | |
| download | op-kernel-dev-4408dd15d9cfcf78b819d2d4a5b43dc36056cab8.zip op-kernel-dev-4408dd15d9cfcf78b819d2d4a5b43dc36056cab8.tar.gz | |
Bluetooth: Use memdup_user in le_auto_conn_write()
This patch does a small code simplification replacing the tipical
kmalloc-copy_from_user sequence by memdup_user() helper.
Cc: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: Andre Guedes <andre.guedes@openbossa.org>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
| -rw-r--r-- | net/bluetooth/hci_core.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c index 1c6ffaa..bfc6f81 100644 --- a/net/bluetooth/hci_core.c +++ b/net/bluetooth/hci_core.c @@ -955,14 +955,9 @@ static ssize_t le_auto_conn_write(struct file *file, const char __user *data, if (count < 3) return -EINVAL; - buf = kzalloc(count, GFP_KERNEL); - if (!buf) - return -ENOMEM; - - if (copy_from_user(buf, data, count)) { - err = -EFAULT; - goto done; - } + buf = memdup_user(data, count); + if (IS_ERR(buf)) + return PTR_ERR(buf); if (memcmp(buf, "add", 3) == 0) { n = sscanf(&buf[4], "%hhx:%hhx:%hhx:%hhx:%hhx:%hhx %hhu %hhu", |
