diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2017-06-03 21:50:46 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2017-06-04 13:57:24 -0400 |
commit | 5a5011936ec0ef22e9433cf2ee3520db4993b445 (patch) | |
tree | 8589b4dd00e61ab4299c33ed1df31194e9cca32c | |
parent | 31acd2665b8f45548e1acfb9131657646333f91c (diff) |
adb: get rid of pointless access_ok()
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
-rw-r--r-- | drivers/macintosh/adb.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/macintosh/adb.c b/drivers/macintosh/adb.c index fee939efc4fc..039dc8285fc5 100644 --- a/drivers/macintosh/adb.c +++ b/drivers/macintosh/adb.c | |||
@@ -723,8 +723,6 @@ static ssize_t adb_read(struct file *file, char __user *buf, | |||
723 | return -EINVAL; | 723 | return -EINVAL; |
724 | if (count > sizeof(req->reply)) | 724 | if (count > sizeof(req->reply)) |
725 | count = sizeof(req->reply); | 725 | count = sizeof(req->reply); |
726 | if (!access_ok(VERIFY_WRITE, buf, count)) | ||
727 | return -EFAULT; | ||
728 | 726 | ||
729 | req = NULL; | 727 | req = NULL; |
730 | spin_lock_irqsave(&state->lock, flags); | 728 | spin_lock_irqsave(&state->lock, flags); |
@@ -781,8 +779,6 @@ static ssize_t adb_write(struct file *file, const char __user *buf, | |||
781 | return -EINVAL; | 779 | return -EINVAL; |
782 | if (adb_controller == NULL) | 780 | if (adb_controller == NULL) |
783 | return -ENXIO; | 781 | return -ENXIO; |
784 | if (!access_ok(VERIFY_READ, buf, count)) | ||
785 | return -EFAULT; | ||
786 | 782 | ||
787 | req = kmalloc(sizeof(struct adb_request), | 783 | req = kmalloc(sizeof(struct adb_request), |
788 | GFP_KERNEL); | 784 | GFP_KERNEL); |