From: Julien Grall <julien.grall@citrix.com>
Date: Thu, 13 Aug 2015 11:03:43 +0000 (+0100)
Subject: xen/arm: mm: Do not dump the p2m when mapping a foreign gfn
X-Git-Tag: archive/raspbian/4.8.0-1+rpi1~1^2~2595
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=afc13fe5e21d18c09e44f8ae6f7f4484e9f1de7f;p=xen.git

xen/arm: mm: Do not dump the p2m when mapping a foreign gfn

The physmap operation XENMAPSPACE_gfmn_foreign is dumping the p2m when
an error occured by calling dump_p2m_lookup. But this function is not
using ratelimited printk.

Any domain able to map foreign gfmn would be able to flood the Xen
console.

The information wasn't not useful so drop it.

This is XSA-141.

Signed-off-by: Julien Grall <julien.grall@citrix.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
---

diff --git a/xen/arch/arm/mm.c b/xen/arch/arm/mm.c
index d6f64cc97f..b5d8c856d6 100644
--- a/xen/arch/arm/mm.c
+++ b/xen/arch/arm/mm.c
@@ -1114,7 +1114,6 @@ int xenmem_add_to_physmap_one(
         page = get_page_from_gfn(od, idx, &p2mt, P2M_ALLOC);
         if ( !page )
         {
-            dump_p2m_lookup(od, pfn_to_paddr(idx));
             rcu_unlock_domain(od);
             return -EINVAL;
         }