From afc13fe5e21d18c09e44f8ae6f7f4484e9f1de7f Mon Sep 17 00:00:00 2001 From: Julien Grall Date: Thu, 13 Aug 2015 12:03:43 +0100 Subject: [PATCH] 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 Acked-by: Ian Campbell --- xen/arch/arm/mm.c | 1 - 1 file changed, 1 deletion(-) 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; } -- 2.30.2