Skip to content

Commit f79aa73

Browse files
committed
Ismail caught a logic error on my part, used a built-in method
instead.
1 parent d4125c1 commit f79aa73

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

lldb/source/Target/Process.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6186,10 +6186,8 @@ Status Process::GetMemoryRegionInfo(lldb::addr_t load_addr,
61866186
load_addr = abi->FixAnyAddress(load_addr);
61876187
Status error = DoGetMemoryRegionInfo(load_addr, range_info);
61886188
// Reject a region that does not contain the requested address.
6189-
if (error.Success() && (range_info.GetRange().GetRangeBase() < load_addr ||
6190-
range_info.GetRange().GetRangeEnd() <= load_addr))
6191-
error =
6192-
Status::FromErrorString("Invalid memory region");
6189+
if (error.Success() && !range_info.GetRange().Contains(load_addr))
6190+
error = Status::FromErrorString("Invalid memory region");
61936191

61946192
return error;
61956193
}

0 commit comments

Comments
 (0)