From: Jiaxun Yang <jiaxun.y...@flygoat.com> >>> CID 1547264: Null pointer dereferences (REVERSE_INULL) >>> Null-checking "ipi" suggests that it may be null, but it has already >>> been dereferenced on all paths leading to the check.
Resolves: Coverity CID 1547264 Link: https://lore.kernel.org/qemu-devel/752417ad-ab72-4fed-8d1f-af41f15bc...@app.fastmail.com/ Signed-off-by: Jiaxun Yang <jiaxun.y...@flygoat.com> Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org> Message-ID: <20240621-loongson3-ipi-follow-v2-2-848eafcbb...@flygoat.com> Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org> --- hw/mips/loongson3_virt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/mips/loongson3_virt.c b/hw/mips/loongson3_virt.c index 27a85e3614..2067b4fecb 100644 --- a/hw/mips/loongson3_virt.c +++ b/hw/mips/loongson3_virt.c @@ -574,7 +574,7 @@ static void mips_loongson3_virt_init(MachineState *machine) cpu_mips_clock_init(cpu); qemu_register_reset(main_cpu_reset, cpu); - if (ipi) { + if (!kvm_enabled()) { hwaddr base = ((hwaddr)node << 44) + virt_memmap[VIRT_IPI].base; base += core * 0x100; qdev_connect_gpio_out(ipi, i, cpu->env.irq[6]); -- 2.45.2