From eaf83aa6b424556bed5c34b97c569acd82a29bdb Mon Sep 17 00:00:00 2001 From: Ryan Houdek Date: Thu, 29 Feb 2024 04:02:05 -0800 Subject: [PATCH] Fix reserving range check Fixes an issue where TestHarnessRunner was managing to reserve the space below stack again, resulting in stack growth breaking. Would typically only show up when using the vixl simulator under gdb for some reason. This is likely the last bandage on this code before it gets completely rewritten to be more readable. --- FEXCore/Source/Utils/Allocator.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/FEXCore/Source/Utils/Allocator.cpp b/FEXCore/Source/Utils/Allocator.cpp index d3b7066f6..10fb13328 100644 --- a/FEXCore/Source/Utils/Allocator.cpp +++ b/FEXCore/Source/Utils/Allocator.cpp @@ -297,7 +297,7 @@ namespace FEXCore::Allocator { if (c == ' ') { STEAL_LOG("[%d] ParseEnd; RegionBegin: %016lX RegionEnd: %016lX\n", __LINE__, RegionBegin, RegionEnd); - if (RegionEnd >= End) { + if (RegionEnd > End) { // Early return if we are completely beyond the allocation space. close(MapsFD); return Regions;