vhost: fix total length when packets are too short
authorMichael S. Tsirkin <mst@redhat.com>
Thu, 27 Mar 2014 10:00:26 +0000 (12:00 +0200)
committerJiri Slaby <jslaby@suse.cz>
Fri, 18 Apr 2014 09:07:14 +0000 (11:07 +0200)
commitc66635949a08858e6d588b8a32489bca7972cf87
treeca1bd83181ce12f37a8d9a1ab8e1f7989ec806a7
parent9ef39f43c0380ea34c0f2d8fc669a81cebd91d4e
vhost: fix total length when packets are too short

[ Upstream commit d8316f3991d207fe32881a9ac20241be8fa2bad0 ]

When mergeable buffers are disabled, and the
incoming packet is too large for the rx buffer,
get_rx_bufs returns success.

This was intentional in order for make recvmsg
truncate the packet and then handle_rx would
detect err != sock_len and drop it.

Unfortunately we pass the original sock_len to
recvmsg - which means we use parts of iov not fully
validated.

Fix this up by detecting this overrun and doing packet drop
immediately.

CVE-2014-0077

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
drivers/vhost/net.c