[xen staging-4.18] x86/ioremap: prevent additions against the NULL pointer

[patchbot@xxxxxxx]

commit ca799f1baafc059a9d8708e7e6ca9ea4f8e893ed
Author:     Roger Pau Monné <roger.pau@xxxxxxxxxx>
AuthorDate: Thu Mar 20 13:35:04 2025 +0100
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Thu Mar 20 13:35:04 2025 +0100

    x86/ioremap: prevent additions against the NULL pointer
    
    This was reported by clang UBSAN as:
    
    UBSAN: Undefined behaviour in arch/x86/mm.c:6297:40
    applying zero offset to null pointer
    [...]
    Xen call trace:
        [<ffff82d040303662>] R common/ubsan/ubsan.c#ubsan_epilogue+0xa/0xc0
        [<ffff82d040304aa3>] F __ubsan_handle_pointer_overflow+0xcb/0x100
        [<ffff82d0406ebbc0>] F ioremap_wc+0xc8/0xe0
        [<ffff82d0406c3728>] F video_init+0xd0/0x180
        [<ffff82d0406ab6f5>] F console_init_preirq+0x3d/0x220
        [<ffff82d0406f1876>] F __start_xen+0x68e/0x5530
        [<ffff82d04020482e>] F __high_start+0x8e/0x90
    
    Fix bt_ioremap() and ioremap{,_wc}() to not add the offset if the returned
    pointer from __vmap() is NULL.
    
    Fixes: d0d4635d034f ('implement vmap()')
    Fixes: f390941a92f1 ('x86/DMI: fix table mapping when one lives above 1Mb')
    Fixes: 81d195c6c0e2 ('x86: introduce ioremap_wc()')
    Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
    Reviewed-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    master commit: 9a6f2c52f75781acda39fab5cc96d1bcc54bf534
    master date: 2025-03-17 13:33:29 +0100
---
 xen/arch/x86/dmi_scan.c | 7 +++++--
 xen/arch/x86/mm.c       | 6 ++++--
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/xen/arch/x86/dmi_scan.c b/xen/arch/x86/dmi_scan.c
index 81f80c053a..b517c068b8 100644
--- a/xen/arch/x86/dmi_scan.c
+++ b/xen/arch/x86/dmi_scan.c
@@ -113,6 +113,7 @@ static const void *__init bt_ioremap(paddr_t addr, unsigned 
int len)
 {
     mfn_t mfn = _mfn(PFN_DOWN(addr));
     unsigned int offs = PAGE_OFFSET(addr);
+    void *va;
 
     if ( addr + len <= MB(1) )
         return __va(addr);
@@ -120,8 +121,10 @@ static const void *__init bt_ioremap(paddr_t addr, 
unsigned int len)
     if ( system_state < SYS_STATE_boot )
         return __acpi_map_table(addr, len);
 
-    return __vmap(&mfn, PFN_UP(offs + len), 1, 1, PAGE_HYPERVISOR_RO,
-                  VMAP_DEFAULT) + offs;
+    va = __vmap(&mfn, PFN_UP(offs + len), 1, 1, PAGE_HYPERVISOR_RO,
+                VMAP_DEFAULT);
+
+    return va ? va + offs : NULL;
 }
 
 static void __init bt_iounmap(const void *ptr, unsigned int len)
diff --git a/xen/arch/x86/mm.c b/xen/arch/x86/mm.c
index 2f0e586463..1454a18fc2 100644
--- a/xen/arch/x86/mm.c
+++ b/xen/arch/x86/mm.c
@@ -5994,7 +5994,9 @@ void __iomem *ioremap(paddr_t pa, size_t len)
         unsigned int offs = pa & (PAGE_SIZE - 1);
         unsigned int nr = PFN_UP(offs + len);
 
-        va = __vmap(&mfn, nr, 1, 1, PAGE_HYPERVISOR_UCMINUS, VMAP_DEFAULT) + 
offs;
+        va = __vmap(&mfn, nr, 1, 1, PAGE_HYPERVISOR_UCMINUS, VMAP_DEFAULT);
+        if ( va )
+            va += offs;
     }
 
     return (void __force __iomem *)va;
@@ -6011,7 +6013,7 @@ void __iomem *__init ioremap_wc(paddr_t pa, size_t len)
 
     va = __vmap(&mfn, nr, 1, 1, PAGE_HYPERVISOR_WC, VMAP_DEFAULT);
 
-    return (void __force __iomem *)(va + offs);
+    return (void __force __iomem *)(va ? va + offs : NULL);
 }
 
 int create_perdomain_mapping(struct domain *d, unsigned long va,
--
generated by git-patchbot for /home/xen/git/xen.git#staging-4.18