vhost: potential crash in map_guest_mem using debug image 35/18235/2
authorSteven Luong <sluong@cisco.com>
Thu, 24 Jan 2019 06:20:19 +0000 (22:20 -0800)
committerDamjan Marion <dmarion@me.com>
Wed, 13 Mar 2019 08:23:25 +0000 (08:23 +0000)
commitec950c23a7e3ac7bd46653addc3cca294b2daa6a
tree804296f017597c6c6e5a401576aae5bce290763f
parent39382976701926c1f34191c1311829c15a53cb01
vhost: potential crash in map_guest_mem using debug image

map_guest_mem may be called from worker-thread/dataplane. It has a call
to vlib_log and may crash inside vlib_log's ASSERT statement

  /* make sure we are running on the main thread to avoid use in dataplane
     code, for dataplane logging consider use of event-logger */
  ASSERT (vlib_get_thread_index () == 0);

The fix is to convert the vlib_log call in map_guest_map to event logger

Change-Id: Iaaf6d86782aa8a18d25e0209f22dc31f04668d56
Signed-off-by: Steven Luong <sluong@cisco.com>
(cherry picked from commit b0789106cb4f4c7ac529c02a5ff1ac346f0913dd)
src/vnet/devices/virtio/vhost_user_inline.h