vhost: buffers leak and interface disable upon vring descriptor out of mmap 17/6817/1 v17.04.1
authorSteven <[email protected]>
Fri, 19 May 2017 04:22:00 +0000 (21:22 -0700)
committerJohn Lo <[email protected]>
Sat, 20 May 2017 17:50:58 +0000 (17:50 +0000)
commit7d68ec61343800f5b2d0660265b02e12b19cb640
tree410a3c31523ae8b59817d3618a9a899ca4be96e2
parent7ebd374e182275aed72860ba0fff6da5af8b53d1
vhost: buffers leak and interface disable upon vring descriptor out of mmap

When processing a vring descriptor which is outside of mmap, we disable
the interface and spit a message to shut/no shut the interface. This is
not practical as application using vhost cannot constantly checking the
logs and do the recovery. The proposed fix is to log an error, like
other errors that we encounter.

The other bug is buffer leak in the function rewind. At the end of the
while loop when b_current != b_head, we still have to give back 1 more
buffer or add 1 to rx_buffers_len.

Change-Id: I68c0b24f070e644cd8878f42272a7b518f14393f
Signed-off-by: Steven <[email protected]>
(cherry picked from commit 95827e430762a2858f4e56e1248a4a93d629a938)
src/vnet/devices/virtio/vhost-user.c