From: Wei Liu Date: Tue, 20 Oct 2020 13:14:38 +0000 (+0200) Subject: x86/mm: Refactor modify_xen_mappings to have one exit path X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=f5ec9f287d00af28b2939c0f628c25c31ebefd18;p=xen.git x86/mm: Refactor modify_xen_mappings to have one exit path We will soon need to perform clean-ups before returning. No functional change. This is part of XSA-345. Reported-by: Hongyan Xia Signed-off-by: Wei Liu Signed-off-by: Hongyan Xia Signed-off-by: George Dunlap Acked-by: Jan Beulich master commit: b733f8a8b8db83f2d438cab3adb38b387cecfce0 master date: 2020-10-20 14:19:55 +0200 --- diff --git a/xen/arch/x86/mm.c b/xen/arch/x86/mm.c index 79a3fac3cc..8ed3ecacbe 100644 --- a/xen/arch/x86/mm.c +++ b/xen/arch/x86/mm.c @@ -5577,6 +5577,7 @@ int modify_xen_mappings(unsigned long s, unsigned long e, unsigned int nf) l1_pgentry_t *pl1e; unsigned int i; unsigned long v = s; + int rc = -ENOMEM; /* Set of valid PTE bits which may be altered. */ #define FLAGS_MASK (_PAGE_NX|_PAGE_RW|_PAGE_PRESENT) @@ -5618,7 +5619,8 @@ int modify_xen_mappings(unsigned long s, unsigned long e, unsigned int nf) /* PAGE1GB: shatter the superpage and fall through. */ pl2e = alloc_xen_pagetable(); if ( !pl2e ) - return -ENOMEM; + goto out; + for ( i = 0; i < L2_PAGETABLE_ENTRIES; i++ ) l2e_write(pl2e + i, l2e_from_pfn(l3e_get_pfn(*pl3e) + @@ -5673,7 +5675,8 @@ int modify_xen_mappings(unsigned long s, unsigned long e, unsigned int nf) /* PSE: shatter the superpage and try again. */ pl1e = alloc_xen_pagetable(); if ( !pl1e ) - return -ENOMEM; + goto out; + for ( i = 0; i < L1_PAGETABLE_ENTRIES; i++ ) l1e_write(&pl1e[i], l1e_from_pfn(l2e_get_pfn(*pl2e) + i, @@ -5802,7 +5805,10 @@ int modify_xen_mappings(unsigned long s, unsigned long e, unsigned int nf) flush_area(NULL, FLUSH_TLB_GLOBAL); #undef FLAGS_MASK - return 0; + rc = 0; + + out: + return rc; } #undef flush_area