KVM: VMX: fix page leak in hardware_setup()

jsmattsonjr · rkrcmar · commit 2895db67b01c · 2017-12-05T22:34:49.000+01:00
vmx_io_bitmap_b should not be allocated twice. Fixes: 2361133 ("KVM: VMX: refactor setup of global page-sized bitmaps") Signed-off-by: Jim Mattson <jmattson@google.com> Reviewed-by: Krish Sadhukhan <krish.sadhukhan@oracle.com> Reviewed-by: David Hildenbrand <david@redhat.com> Signed-off-by: Radim Krčmář <rkrcmar@redhat.com>
diff --git a/arch/x86/kvm/vmx.c b/arch/x86/kvm/vmx.c
@@ -6751,7 +6751,6 @@ static __init int hardware_setup(void)
 			goto out;
 	}
 
-	vmx_io_bitmap_b = (unsigned long *)__get_free_page(GFP_KERNEL);
 	memset(vmx_vmread_bitmap, 0xff, PAGE_SIZE);
 	memset(vmx_vmwrite_bitmap, 0xff, PAGE_SIZE);
 

Original file line number	Diff line number	Diff line change
`@@ -6751,7 +6751,6 @@ static __init int hardware_setup(void)`
`6751`	`6751`	`goto out;`
`6752`	`6752`	`}`
`6753`	`6753`
`6754`		`- vmx_io_bitmap_b = (unsigned long *)__get_free_page(GFP_KERNEL);`
`6755`	`6754`	`memset(vmx_vmread_bitmap, 0xff, PAGE_SIZE);`
`6756`	`6755`	`memset(vmx_vmwrite_bitmap, 0xff, PAGE_SIZE);`
`6757`	`6756`