dma-debug: fix off-by-one error in overlap function

This patch fixes a bug in the overlap function which returned true if
one region ends exactly before the second region begins. This is no
overlap but the function returned true in that case.

Cc: [email protected]
Reported-by: Andrew Randrianasulu <[email protected]>
Signed-off-by: Joerg Roedel <[email protected]>

diff --git a/lib/dma-debug.c b/lib/dma-debug.c
index 3b93129a968c73b1d6eeff421abb76b38bc668dd..a9b6b5c9e091680479b51a496cb6aa1e41fabd1b 100644 (file)
--- a/lib/dma-debug.c
+++ b/lib/dma-debug.c
@@ -862,7 +862,7 @@ static inline bool overlap(void *addr, u64 size, void *start, void *end)
 
        return ((addr >= start && addr < end) ||
                (addr2 >= start && addr2 < end) ||
-               ((addr < start) && (addr2 >= end)));
+               ((addr < start) && (addr2 > end)));
 }
 
 static void check_for_illegal_area(struct device *dev, void *addr, u64 size)