Fix: treat kernel_slice_end as an exclusive bound when checking for overlaps

phil-opp · phil-opp · commit 8b64b209f11c · 2023-01-26T11:57:27.000+01:00
Fixes an error that occured when the exclusive kernel end address was equal to the start address of the next memory region. We erroneously considered these regions as overlapping.
diff --git a/common/src/legacy_memory_region.rs b/common/src/legacy_memory_region.rs
@@ -160,7 +160,7 @@ where
             let kernel_slice_end = kernel_slice_start + kernel_slice_len;
             if region.kind == MemoryRegionKind::Usable
                 && kernel_slice_start < region.end
-                && kernel_slice_end >= region.start
+                && kernel_slice_end > region.start
             {
                 // region overlaps with kernel -> we might need to split it
 

Original file line number	Diff line number	Diff line change
`@@ -160,7 +160,7 @@ where`
`160`	`160`	`let kernel_slice_end = kernel_slice_start + kernel_slice_len;`
`161`	`161`	`if region.kind == MemoryRegionKind::Usable`
`162`	`162`	`&& kernel_slice_start < region.end`
`163`		`- && kernel_slice_end >= region.start`
	`163`	`+ && kernel_slice_end > region.start`
`164`	`164`	`{`
`165`	`165`	`// region overlaps with kernel -> we might need to split it`
`166`	`166`