[PATCH net-next 14/19] batadv_socket_read(): get rid of pointless access_ok()

From: Al Viro
Date: Wed May 20 2020 - 20:38:12 EST


From: Al Viro <viro@xxxxxxxxxxxxxxxxxx>

address is passed only to copy_to_user()

Signed-off-by: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
---
net/batman-adv/icmp_socket.c | 3 ---
1 file changed, 3 deletions(-)

diff --git a/net/batman-adv/icmp_socket.c b/net/batman-adv/icmp_socket.c
index ccb535c77e5d..8bdabc03b0b2 100644
--- a/net/batman-adv/icmp_socket.c
+++ b/net/batman-adv/icmp_socket.c
@@ -135,9 +135,6 @@ static ssize_t batadv_socket_read(struct file *file, char __user *buf,
if (!buf || count < sizeof(struct batadv_icmp_packet))
return -EINVAL;

- if (!access_ok(buf, count))
- return -EFAULT;
-
error = wait_event_interruptible(socket_client->queue_wait,
socket_client->queue_len);

--
2.11.0