[Xen-changelog] [xen stable-4.4] xen/arm: Don't free p2m->first_level in p2m_teardown() before it has been allocated

[patchbot@xxxxxxx]

commit 01311b90ced670a8025690eaa13ab461d1bc5e67
Author:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
AuthorDate: Thu Jun 2 14:19:00 2016 +0100
Commit:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
CommitDate: Fri Jun 3 11:31:25 2016 +0100

    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 <Aaron.Cornelius@xxxxxxxxxxxxxxx>
    Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
    Reviewed-by: Julien Grall <julien.grall@xxxxxxx>
---
 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;
 
--
generated by git-patchbot for /home/xen/git/xen.git#stable-4.4

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog