diff options
author | Jason Wang <jasowang@redhat.com> | 2019-04-09 00:10:25 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-04-11 01:45:38 -0400 |
commit | 813dbeb656d6c90266f251d8bd2b02d445afa63f (patch) | |
tree | 24909b6cd09af83979b11b7e7b6f5b7cc2d2ba35 /drivers/vhost/vhost.c | |
parent | b4f47f3848eb70986f75d06112af7b48b7f5f462 (diff) |
vhost: reject zero size iova range
We used to accept zero size iova range which will lead a infinite loop
in translate_desc(). Fixing this by failing the request in this case.
Reported-by: syzbot+d21e6e297322a900c128@syzkaller.appspotmail.com
Fixes: 6b1e6cc7 ("vhost: new device IOTLB API")
Signed-off-by: Jason Wang <jasowang@redhat.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/vhost/vhost.c')
-rw-r--r-- | drivers/vhost/vhost.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c index 5ace833de746..351af88231ad 100644 --- a/drivers/vhost/vhost.c +++ b/drivers/vhost/vhost.c | |||
@@ -911,8 +911,12 @@ static int vhost_new_umem_range(struct vhost_umem *umem, | |||
911 | u64 start, u64 size, u64 end, | 911 | u64 start, u64 size, u64 end, |
912 | u64 userspace_addr, int perm) | 912 | u64 userspace_addr, int perm) |
913 | { | 913 | { |
914 | struct vhost_umem_node *tmp, *node = kmalloc(sizeof(*node), GFP_ATOMIC); | 914 | struct vhost_umem_node *tmp, *node; |
915 | 915 | ||
916 | if (!size) | ||
917 | return -EFAULT; | ||
918 | |||
919 | node = kmalloc(sizeof(*node), GFP_ATOMIC); | ||
916 | if (!node) | 920 | if (!node) |
917 | return -ENOMEM; | 921 | return -ENOMEM; |
918 | 922 | ||