[TSan] [Darwin] Fix off by one in TSAN init due to MemoryRangeIsAvailable (#169008)

ndrewh · web-flow · commit bb2e4686c1c8 · 2025-11-21T07:47:11.000-08:00
diff --git a/compiler-rt/lib/tsan/rtl/tsan_platform_mac.cpp b/compiler-rt/lib/tsan/rtl/tsan_platform_mac.cpp
@@ -235,7 +235,7 @@ void InitializePlatformEarly() {
   }
   // In some configurations, the max_vm is expanded, but much of this space is
   // already mapped. TSAN will not work in this configuration.
-  if (!MemoryRangeIsAvailable(HiAppMemEnd() - 1, HiAppMemEnd())) {
+  if (!MemoryRangeIsAvailable(HiAppMemEnd() - 1, HiAppMemEnd() - 1)) {
     Report(
         "ThreadSanitizer: Unsupported virtual memory layout: Address %p is "
         "already mapped.\n",

Original file line number	Diff line number	Diff line change
`@@ -235,7 +235,7 @@ void InitializePlatformEarly() {`
`235`	`235`	`}`
`236`	`236`	`// In some configurations, the max_vm is expanded, but much of this space is`
`237`	`237`	`// already mapped. TSAN will not work in this configuration.`
`238`		`- if (!MemoryRangeIsAvailable(HiAppMemEnd() - 1, HiAppMemEnd())) {`
	`238`	`+ if (!MemoryRangeIsAvailable(HiAppMemEnd() - 1, HiAppMemEnd() - 1)) {`
`239`	`239`	`Report(`
`240`	`240`	`"ThreadSanitizer: Unsupported virtual memory layout: Address %p is "`
`241`	`241`	`"already mapped.\n",`