diff options
author | Liran Alon <liran.alon@oracle.com> | 2018-09-16 14:28:20 +0300 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2018-09-20 18:54:08 +0200 |
commit | 26b471c7e2f7befd0f59c35b257749ca57e0ed70 (patch) | |
tree | 955a3c744216141bc2beb4e4e93655e3dec37598 /lib/crc32defs.h | |
parent | 8b56ee91ffc88ea01400c012e10fe22a9d233265 (diff) |
KVM: nVMX: Fix bad cleanup on error of get/set nested state IOCTLs
The handlers of IOCTLs in kvm_arch_vcpu_ioctl() are expected to set
their return value in "r" local var and break out of switch block
when they encounter some error.
This is because vcpu_load() is called before the switch block which
have a proper cleanup of vcpu_put() afterwards.
However, KVM_{GET,SET}_NESTED_STATE IOCTLs handlers just return
immediately on error without performing above mentioned cleanup.
Thus, change these handlers to behave as expected.
Fixes: 8fcc4b5923af ("kvm: nVMX: Introduce KVM_CAP_NESTED_STATE")
Reviewed-by: Mark Kanda <mark.kanda@oracle.com>
Reviewed-by: Patrick Colp <patrick.colp@oracle.com>
Signed-off-by: Liran Alon <liran.alon@oracle.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'lib/crc32defs.h')
0 files changed, 0 insertions, 0 deletions