On 3/12/25 09:39, Markus Armbruster wrote:
scripts/coccinelle/error-use-after-free.cocci led me to target/loongarch/kvm/kvm.c:ret = kvm_cpu_check_lsx(cs, &local_err); if (ret < 0) { error_report_err(local_err); Reporting an error, but continue anyway. This is suspicious. } ret = kvm_cpu_check_lasx(cs, &local_err); Passing non-null @local_error to kvm_cpu_check_lasx(). This is wrong. When kvm_cpu_check_lasx() fails and passes &local_error to error_setg(), error_setv()'s assertion will fail. Two possible fixes: 1. If continuing after kvm_cpu_check_lasx() failure is correct, we need to clear @local_error there. Since it's not actually an error then, we should almost certainly not use error_report_err() there. *Maybe* warn_report_err(). 2. If continuing is wrong, we probably need to return ret.
Indeed the correct fix is to return ret, since the Error is set whenever an OnOffAuto property is "on" and KVM does not support a feature.
Same for all those below. Paolo
