Commit 22d7108c authored Aug 26, 2021 by Thomas Huth Committed by Paolo Bonzini Sep 30, 2021

KVM: selftests: Fix kvm_vm_free() in cr4_cpuid_sync and vmx_tsc_adjust tests



The kvm_vm_free() statement here is currently dead code, since the loop
in front of it can only be left with the "goto done" that jumps right
after the kvm_vm_free(). Fix it by swapping the locations of the "done"
label and the kvm_vm_free().

Signed-off-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20210826074928.240942-1-thuth@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

parent d22869af

tools/testing/selftests/kvm/x86_64/cr4_cpuid_sync_test.c

+1 −2

Original line number	Diff line number	Diff line
		@@ -109,8 +109,7 @@ int main(int argc, char *argv[])
		}
		}

		kvm_vm_free(vm);

		done:
		kvm_vm_free(vm);
		return 0;
		}

tools/testing/selftests/kvm/x86_64/vmx_tsc_adjust_test.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -161,7 +161,7 @@ int main(int argc, char *argv[])
		}
		}

		kvm_vm_free(vm);
		done:
		kvm_vm_free(vm);
		return 0;
		}