Vitaly Kuznetsov d5cd6f3401 KVM: nSVM: Avoid freeing uninitialized pointers in svm_set_nested_state() ... The save and ctl pointers are passed uninitialized to kfree() when svm_set_nested_state() follows the 'goto out_set_gif' path. While the issue could've been fixed by initializing these on-stack varialbles to NULL, it seems preferable to eliminate 'out_set_gif' label completely as it is not actually a failure path and duplicating a single svm_set_gif() call doesn't look too bad. [ bp: Drop obscure Addresses-Coverity: tag. ] Fixes: 6ccbd29ade ("KVM: SVM: nested: Don't allocate VMCB structures on stack") Reported-by: Dan Carpenter <dan.carpenter@oracle.com> Reported-by: Joerg Roedel <jroedel@suse.de> Reported-by: Colin King <colin.king@canonical.com> Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com> Signed-off-by: Borislav Petkov <bp@suse.de> Reviewed-by: Sean Christopherson <sean.j.christopherson@intel.com> Acked-by: Joerg Roedel <jroedel@suse.de> Tested-by: Tom Lendacky <thomas.lendacky@amd.com> Link: https://lkml.kernel.org/r/20200914133725.650221-1-vkuznets@redhat.com Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>		2020-09-28 07:56:43 -04:00
..
avic.c	KVM: SVM: Add vmcb_ prefix to mark_*() functions	2020-07-08 16:21:48 -04:00
nested.c	KVM: nSVM: Avoid freeing uninitialized pointers in svm_set_nested_state()	2020-09-28 07:56:43 -04:00
pmu.c	KVM: x86/pmu: Tweak kvm_pmu_get_msr to pass 'struct msr_data' in	2020-06-01 04:26:08 -04:00
sev.c	KVM: SVM: Periodically schedule when unregistering regions on destroy	2020-09-11 13:24:15 -04:00
svm.c	Merge branch 'x86-seves-for-paolo' of https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip into HEAD	2020-09-22 06:43:17 -04:00
svm.h	KVM: nSVM: prepare to handle errors from enter_svm_guest_mode()	2020-07-10 12:55:13 -04:00
vmenter.S	x86/kvm/svm: Move guest enter/exit into .noinstr.text	2020-07-09 07:08:41 -04:00