As clang rightfully pointed out, the ampersand in front of this member looks wrong. Remove it so we actually really compare against the count being 0. Signed-off-by: Andre Przywara <andre.przywara@xxxxxxx> Reviewed-by: Jean-Philippe Brucker <jean-philippe.brucker@xxxxxxx> --- vfio/pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/vfio/pci.c b/vfio/pci.c index f17498ea..10aa87b1 100644 --- a/vfio/pci.c +++ b/vfio/pci.c @@ -952,7 +952,7 @@ static int vfio_pci_init_msis(struct kvm *kvm, struct vfio_device *vdev, size_t nr_entries = msis->nr_entries; ret = ioctl(vdev->fd, VFIO_DEVICE_GET_IRQ_INFO, &msis->info); - if (ret || &msis->info.count == 0) { + if (ret || msis->info.count == 0) { vfio_dev_err(vdev, "no MSI reported by VFIO"); return -ENODEV; } -- 2.17.1