IsaIrqState::ioapic is always non-NULL. Probably, the concrete qemu_irq was supposed to be tested, but that's already done by qemu_set_irq.
Signed-off-by: Jan Kiszka <jan.kis...@siemens.com> --- hw/pc.c | 5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/hw/pc.c b/hw/pc.c index 203627d..a15d165 100644 --- a/hw/pc.c +++ b/hw/pc.c @@ -96,9 +96,8 @@ void isa_irq_handler(void *opaque, int n, int level) if (n < 16) { qemu_set_irq(isa->i8259[n], level); } - if (isa->ioapic) - qemu_set_irq(isa->ioapic[n], level); -}; + qemu_set_irq(isa->ioapic[n], level); +} static void ioport80_write(void *opaque, uint32_t addr, uint32_t data) { -- 1.7.3.4