Re: [Qemu-devel] [PATCH] virtio: check vring descriptor buffer length

2016-07-28 Thread Stefan Hajnoczi
On Wed, Jul 27, 2016 at 09:07:56PM +0530, P J P wrote: > From: Prasad J Pandit > > virtio back end uses set of buffers to facilitate I/O operations. > An infinite loop unfolds in virtqueue_pop() if a buffer was > of zero size. Add check to avoid it. > > Reported-by: Li Qiang > Signed-off-by: Pr

Re: [Qemu-devel] [PATCH] virtio: check vring descriptor buffer length

2016-07-27 Thread Michael S. Tsirkin
On Wed, Jul 27, 2016 at 10:13:04AM -0600, Eric Blake wrote: > On 07/27/2016 09:37 AM, P J P wrote: > > From: Prasad J Pandit > > > > virtio back end uses set of buffers to facilitate I/O operations. > > An infinite loop unfolds in virtqueue_pop() if a buffer was > > of zero size. Add check to avo

Re: [Qemu-devel] [PATCH] virtio: check vring descriptor buffer length

2016-07-27 Thread Eric Blake
On 07/27/2016 09:37 AM, P J P wrote: > From: Prasad J Pandit > > virtio back end uses set of buffers to facilitate I/O operations. > An infinite loop unfolds in virtqueue_pop() if a buffer was > of zero size. Add check to avoid it. > > Reported-by: Li Qiang > Signed-off-by: Prasad J Pandit > -

[Qemu-devel] [PATCH] virtio: check vring descriptor buffer length

2016-07-27 Thread P J P
From: Prasad J Pandit virtio back end uses set of buffers to facilitate I/O operations. An infinite loop unfolds in virtqueue_pop() if a buffer was of zero size. Add check to avoid it. Reported-by: Li Qiang Signed-off-by: Prasad J Pandit --- hw/virtio/virtio.c | 5 + 1 file changed, 5 ins