[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/2] hw/virtio/virtio: Don't allow guests to add
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 1/2] hw/virtio/virtio: Don't allow guests to add/remove queues |
Date: |
Thu, 25 Jul 2013 23:37:22 +0100 |
On 25 July 2013 23:33, Michael S. Tsirkin <address@hidden> wrote:
> On Thu, Jul 25, 2013 at 02:37:42PM +0100, Peter Maydell wrote:
>> A queue size of 0 is used to indicate a nonexistent queue, so
>> don't allow the guest to flip a queue between zero-size and
>> non-zero-size. Don't permit setting of negative queue sizes
>> either.
>>
>> Signed-off-by: Peter Maydell <address@hidden>
>> ---
>> hw/virtio/virtio.c | 10 +++++++---
>> 1 file changed, 7 insertions(+), 3 deletions(-)
>>
>> diff --git a/hw/virtio/virtio.c b/hw/virtio/virtio.c
>> index 09f62c6..d5b0502 100644
>> --- a/hw/virtio/virtio.c
>> +++ b/hw/virtio/virtio.c
>> @@ -673,10 +673,14 @@ hwaddr virtio_queue_get_addr(VirtIODevice *vdev, int n)
>>
>> void virtio_queue_set_num(VirtIODevice *vdev, int n, int num)
>> {
>> - if (num <= VIRTQUEUE_MAX_SIZE) {
>> - vdev->vq[n].vring.num = num;
>> - virtqueue_init(&vdev->vq[n]);
>> + if ((num == 0 && vdev->vq[n].vring.num != 0) ||
>> + (num != 0 && vdev->vq[n].vring.num == 0) ||
>
> Cleaner (imho)
>
> !num != !vdev->vq[n].vring.num
I think that's more confusing, and you really don't want
"guards so we don't let the guest do bad things" to be
confusing to read.
>> + (num < 0)) {
>
> How does it ever get negative?
If the guest maliciously writes a value with bit 31 set
to the register...
-- PMM
[Qemu-devel] [PATCH 2/2] hw/virtio/virtio-mmio: Make QueueNumMax read 0 for unavailable queues, Peter Maydell, 2013/07/25