From 605a2711c411247920116a5026e772815b1168cd Mon Sep 17 00:00:00 2001 From: Andrew Cooper Date: Thu, 2 Jun 2016 14:19:00 +0100 Subject: [PATCH] xen/arm: Don't free p2m->first_level in p2m_teardown() before it has been allocated If p2m_init() didn't complete successfully, (e.g. due to VMID exhaustion), p2m_teardown() is called and unconditionally tries to free p2m->first_level before it has been allocated. free_domheap_pages() doesn't tolerate NULL pointers. This is XSA-181 Reported-by: Aaron Cornelius Signed-off-by: Andrew Cooper Reviewed-by: Jan Beulich Reviewed-by: Julien Grall --- xen/arch/arm/p2m.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/xen/arch/arm/p2m.c b/xen/arch/arm/p2m.c index aff7a2c..9cf6f91 100644 --- a/xen/arch/arm/p2m.c +++ b/xen/arch/arm/p2m.c @@ -615,7 +615,8 @@ void p2m_teardown(struct domain *d) while ( (pg = page_list_remove_head(&p2m->pages)) ) free_domheap_page(pg); - free_domheap_pages(p2m->first_level, P2M_FIRST_ORDER); + if ( p2m->first_level ) + free_domheap_pages(p2m->first_level, P2M_FIRST_ORDER); p2m->first_level = NULL; -- 2.1.4