]> git.baikalelectronics.ru Git - kernel.git/commit
KVM: nVMX: fix nested_vmx_check_vmptr failure paths under debugging
authorRadim Krčmář <rkrcmar@redhat.com>
Fri, 19 May 2017 13:48:51 +0000 (15:48 +0200)
committerPaolo Bonzini <pbonzini@redhat.com>
Tue, 30 May 2017 15:17:21 +0000 (17:17 +0200)
commit290b696a33683a2e495a481aeb3be3c87ca05f37
tree9e3120f4bc4032961a5219b6aed54cbb28a9b03f
parentfe9fa689c7a6a928685d0da4f4f3d4a762c8f4fe
KVM: nVMX: fix nested_vmx_check_vmptr failure paths under debugging

kvm_skip_emulated_instruction() will return 0 if userspace is
single-stepping the guest.

kvm_skip_emulated_instruction() uses return status convention of exit
handler: 0 means "exit to userspace" and 1 means "continue vm entries".
The problem is that nested_vmx_check_vmptr() return status means
something else: 0 is ok, 1 is error.

This means we would continue executing after a failure.  Static checker
noticed it because vmptr was not initialized.

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Fixes: 3f44e781410e ("KVM: x86: Add kvm_skip_emulated_instruction and use it.")
Signed-off-by: Radim Krčmář <rkrcmar@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/x86/kvm/vmx.c