On 6/7/26 14:53, Marc-André Lureau wrote:
tdx_fetch_cpuid() only sets the output ret parameter on the error path. GCC cannot prove that r is always initialized before use in the caller, triggering -Werror=maybe-uninitialized.Initialize r to -1 to silence the warning. Fixes: 228e40f33048 ("i386/tdx: Fetch and validate CPUID of TD guest") Signed-off-by: Marc-Andre Lureau <[email protected]> --- target/i386/kvm/tdx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/target/i386/kvm/tdx.c b/target/i386/kvm/tdx.c index c9c3b05d5fe..8294dbde3aa 100644 --- a/target/i386/kvm/tdx.c +++ b/target/i386/kvm/tdx.c @@ -887,7 +887,7 @@ static int tdx_check_features(X86ConfidentialGuest *cg, CPUState *cs) FeatureWordInfo *wi; FeatureWord w; bool mismatch = false; - int r; + int r = -1;fetch_cpuid = tdx_fetch_cpuid(cs, &r);if (!fetch_cpuid) {
Alternatively, 'return -1;' here. Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
