From: Marc-André Lureau <marcandre.lureau@redhat.com>
Date: Fri, 18 Dec 2015 14:13:08 +0000 (+0100)
Subject: ivshmem: remove redundant assignment, fix crash with msi=off
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=47213eb1104709bf238c8d16db20aa47d37b1c59;p=people%2Fliuw%2Flibxenctrl-split%2Fqemu-xen.git

ivshmem: remove redundant assignment, fix crash with msi=off

Fix crash when msi=false introduced in 660c97ee (msi_vectors is NULL in
this case)

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
---

diff --git a/hw/misc/ivshmem.c b/hw/misc/ivshmem.c
index 358df2407..50297892f 100644
--- a/hw/misc/ivshmem.c
+++ b/hw/misc/ivshmem.c
@@ -355,12 +355,9 @@ static CharDriverState* create_eventfd_chr_device(IVShmemState *s,
                                                   int vector)
 {
     /* create a event character device based on the passed eventfd */
-    PCIDevice *pdev = PCI_DEVICE(s);
     int eventfd = event_notifier_get_fd(n);
     CharDriverState *chr;
 
-    s->msi_vectors[vector].pdev = pdev;
-
     chr = qemu_chr_open_eventfd(eventfd);
 
     if (chr == NULL) {