The following commit has been merged in the master branch:
commit 38c53ca3c114fa2a7030f0d1f54feaf044957609
Author: Al Viro <viro(a)zeniv.linux.org.uk>
Date: Thu Apr 23 10:19:04 2020 -0400
batadv_socket_read(): get rid of pointless access_ok()
address is passed only to copy_to_user()
Signed-off-by: Al Viro <viro(a)zeniv.linux.org.uk>
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);
--
LinuxNextTracking