Re: [Qemu-devel] [PATCH 1/2] virtio-pci: build for uninitialized return value in vq_vector_unmask

[Eduardo Habkost]

On Mon, Jan 14, 2013 at 01:20:12PM -0600, Michael Roth wrote:
> Fixes the following:
> 
> /home/mdroth/w/qemu2.git/hw/virtio-pci.c: In function
> ‘kvm_virtio_pci_vector_unmask’:
> /home/mdroth/w/qemu2.git/hw/virtio-pci.c:673:12: error: ‘ret’ may be
> used uninitialized in this function [-Werror=uninitialized]
> cc1: all warnings being treated as errors
> make: *** [hw/virtio-pci.o] Error 1
> make: *** Waiting for unfinished jobs....
> 
> Signed-off-by: Michael Roth <address@hidden>

Reviewed-by: Eduardo Habkost <address@hidden>

> ---
>  hw/virtio-pci.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/hw/virtio-pci.c b/hw/virtio-pci.c
> index 0b49739..0934246 100644
> --- a/hw/virtio-pci.c
> +++ b/hw/virtio-pci.c
> @@ -616,7 +616,7 @@ static int kvm_virtio_pci_vq_vector_unmask(VirtIOPCIProxy 
> *proxy,
>      VirtQueue *vq = virtio_get_queue(proxy->vdev, queue_no);
>      EventNotifier *n = virtio_queue_get_guest_notifier(vq);
>      VirtIOIRQFD *irqfd = &proxy->vector_irqfd[vector];
> -    int ret;
> +    int ret = 0;
>  
>      if (irqfd->msg.data != msg.data || irqfd->msg.address != msg.address) {
>          ret = kvm_irqchip_update_msi_route(kvm_state, irqfd->virq, msg);
> -- 
> 1.7.9.5
> 
> 

-- 
Eduardo