Commit a63cb560 authored Apr 08, 2013 by Jan Kiszka Committed by Gleb Natapov Apr 08, 2013

KVM: VMX: Add missing braces to avoid redundant error check



The code was already properly aligned, now also add the braces to avoid
that err is checked even if alloc_apic_access_page didn't run and change
it. Found via Coccinelle by Fengguang Wu.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Gleb Natapov <gleb@redhat.com>

parent 458f212e

arch/x86/kvm/vmx.c

+2 −1

Original line number	Diff line number	Diff line
		@@ -6785,10 +6785,11 @@ static struct kvm_vcpu vmx_create_vcpu(struct kvm kvm, unsigned int id)
		put_cpu();
		if (err)
		goto free_vmcs;
		if (vm_need_virtualize_apic_accesses(kvm))
		if (vm_need_virtualize_apic_accesses(kvm)) {
		err = alloc_apic_access_page(kvm);
		if (err)
		goto free_vmcs;
		}

		if (enable_ept) {
		if (!kvm->arch.ept_identity_map_addr)