i386/sev: fix unreachable code coverity issue
authorPankaj Gupta <pankaj.gupta@amd.com>
Fri, 7 Jun 2024 18:36:09 +0000 (13:36 -0500)
committerPaolo Bonzini <pbonzini@redhat.com>
Tue, 11 Jun 2024 12:28:34 +0000 (14:28 +0200)
Set 'finish->id_block_en' early, so that it is properly reset.

Fixes coverity CID 1546887.

Fixes: 7b34df4426 ("i386/sev: Introduce 'sev-snp-guest' object")
Signed-off-by: Pankaj Gupta <pankaj.gupta@amd.com>
Message-ID: <20240607183611.1111100-2-pankaj.gupta@amd.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
target/i386/sev.c

index 004c667ac144851eadccc39b61e68d3c1d374ac2..7c9df621de1162288ed30a0bba810dcef4bf1bf1 100644 (file)
@@ -2165,6 +2165,7 @@ sev_snp_guest_set_id_block(Object *obj, const char *value, Error **errp)
     struct kvm_sev_snp_launch_finish *finish = &sev_snp_guest->kvm_finish_conf;
     gsize len;
 
+    finish->id_block_en = 0;
     g_free(sev_snp_guest->id_block);
     g_free((guchar *)finish->id_block_uaddr);
 
@@ -2184,7 +2185,7 @@ sev_snp_guest_set_id_block(Object *obj, const char *value, Error **errp)
         return;
     }
 
-    finish->id_block_en = (len) ? 1 : 0;
+    finish->id_block_en = 1;
 }
 
 static char *