aboutsummaryrefslogtreecommitdiff
path: root/docs/devel/secure-coding-practices.rst
diff options
context:
space:
mode:
authorRaphael Norwitz <raphael.norwitz@nutanix.com>2019-08-22 11:34:25 -0700
committerMichael S. Tsirkin <mst@redhat.com>2019-09-16 06:27:35 -0400
commitb8487a357855e78957d8c9bddbbcf58548e1aa51 (patch)
tree3f2766084746913848098f68528eddba7f83091d /docs/devel/secure-coding-practices.rst
parentd8438bd24a3f2b6fd6d0e788c18483ca9e239b36 (diff)
downloadqemu-b8487a357855e78957d8c9bddbbcf58548e1aa51.zip
qemu-b8487a357855e78957d8c9bddbbcf58548e1aa51.tar.gz
qemu-b8487a357855e78957d8c9bddbbcf58548e1aa51.tar.bz2
backends/vhost-user.c: prevent using uninitialized vqs
Similar rational to: e6cc11d64fc998c11a4dfcde8fda3fc33a74d844 For vhost scsi and vhost-user-scsi an issue was observed where, of the 3 virtqueues, seabios would only set cmd, leaving ctrl and event without a physical address. This can caused vhost_verify_ring_part_mapping to return ENOMEM, causing the following logs: qemu-system-x86_64: Unable to map available ring for ring 0 qemu-system-x86_64: Verify ring failure on region 0 The issue has already been fixed elsewhere, but it was noted that in backends/vhost-user.c, the vhost_user_backend_dev_init() function, which other vdevs use in their realize() to initialize their vqs, was not being properly zeroing out the queues. This commit ensures hardware modules using the vhost_user_backend_dev_init() API properly zero out their vqs on initialization. Suggested-by: Philippe Mathieu-Daude <philmd@redhat.com> Signed-off-by: Raphael Norwitz <raphael.norwitz@nutanix.com> Message-Id: <1566498865-55506-2-git-send-email-raphael.norwitz@nutanix.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'docs/devel/secure-coding-practices.rst')
0 files changed, 0 insertions, 0 deletions