Commit c8fae27d authored by Li Zhang's avatar Li Zhang Committed by Michael S. Tsirkin

virtio-pci: Check if is_avq is NULL

[bug]
In the virtio_pci_common.c function vp_del_vqs, vp_dev->is_avq is involved
to determine whether it is admin virtqueue, but this function vp_dev->is_avq
 may be empty. For installations, virtio_pci_legacy does not assign a value
 to vp_dev->is_avq.

[fix]
Check whether it is vp_dev->is_avq before use.

[test]
Test with virsh Attach device
Before this patch, the following command would crash the guest system

After applying the patch, everything seems to be working fine.
Signed-off-by: default avatarLi Zhang <zhanglikernel@gmail.com>
Message-Id: <1710566754-3532-1-git-send-email-zhanglikernel@gmail.com>
Signed-off-by: default avatarMichael S. Tsirkin <mst@redhat.com>
parent c0a7233f
...@@ -236,7 +236,7 @@ void vp_del_vqs(struct virtio_device *vdev) ...@@ -236,7 +236,7 @@ void vp_del_vqs(struct virtio_device *vdev)
int i; int i;
list_for_each_entry_safe(vq, n, &vdev->vqs, list) { list_for_each_entry_safe(vq, n, &vdev->vqs, list) {
if (vp_dev->is_avq(vdev, vq->index)) if (vp_dev->is_avq && vp_dev->is_avq(vdev, vq->index))
continue; continue;
if (vp_dev->per_vq_vectors) { if (vp_dev->per_vq_vectors) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment