path: root/net/socket.c
diff options
authorAnton Blanchard <anton@samba.org>2011-08-04 14:07:39 +0000
committerDavid S. Miller <davem@davemloft.net>2011-08-05 03:31:03 -0700
commit98382f419f32d2c12d021943b87dea555677144b (patch)
treed07b99ec8eec119b53ddc2223de9027d7a8c0263 /net/socket.c
parentnet: sendmmsg should only return an error if no messages were sent (diff)
net: Cap number of elements for sendmmsg
To limit the amount of time we can spend in sendmmsg, cap the number of elements to UIO_MAXIOV (currently 1024). For error handling an application using sendmmsg needs to retry at the first unsent message, so capping is simpler and requires less application logic than returning EINVAL. Signed-off-by: Anton Blanchard <anton@samba.org> Cc: stable <stable@kernel.org> [3.0+] Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
1 files changed, 3 insertions, 0 deletions
diff --git a/net/socket.c b/net/socket.c
index e4ed2359eb43..b5c6de4f268a 100644
--- a/net/socket.c
+++ b/net/socket.c
@@ -1999,6 +1999,9 @@ int __sys_sendmmsg(int fd, struct mmsghdr __user *mmsg, unsigned int vlen,
struct compat_mmsghdr __user *compat_entry;
struct msghdr msg_sys;
+ if (vlen > UIO_MAXIOV)
+ vlen = UIO_MAXIOV;
datagrams = 0;
sock = sockfd_lookup_light(fd, &err, &fput_needed);