Update patch to move the check for memory regions to

Process::GetMemoryRegionInfo - I want to reject these
regions from either gdb remote serial protocol stubs,
or scripted processes.

Author: jasonmolenda

lldb/source/Plugins/Process/scripted/ScriptedProcess.cpp

@@ -288,17 +288,8 @@ Status ScriptedProcess::DoGetMemoryRegionInfo(lldb::addr_t load_addr,
                                               MemoryRegionInfo &region) {
   Status error;
   if (auto region_or_err =
-          GetInterface().GetMemoryRegionContainingAddress(load_addr, error)) {
+          GetInterface().GetMemoryRegionContainingAddress(load_addr, error))
     region = *region_or_err;
-    // Reject a region of {0,0} or {0,UINT64_MAX}, neither are
-    // meaningful responses.
-    if (region.GetRange().GetRangeBase() == 0 &&
-        (region.GetRange().GetByteSize() == 0 ||
-         region.GetRange().GetByteSize() == UINT64_MAX)) {
-      error = Status::FromErrorString(
-          "Invalid memory region from scripted process");
-    }
-  }
 
   return error;
 }

lldb/source/Target/Process.cpp

@@ -6184,7 +6184,16 @@ Status Process::GetMemoryRegionInfo(lldb::addr_t load_addr,
                                     MemoryRegionInfo &range_info) {
   if (const lldb::ABISP &abi = GetABI())
     load_addr = abi->FixAnyAddress(load_addr);
-  return DoGetMemoryRegionInfo(load_addr, range_info);
+  Status error = DoGetMemoryRegionInfo(load_addr, range_info);
+  // Reject a region of {0,0} or {0,UINT64_MAX}, neither are
+  // meaningful responses.
+  if (error.Success() && range_info.GetRange().GetRangeBase() == 0 &&
+      (range_info.GetRange().GetByteSize() == 0 ||
+       range_info.GetRange().GetByteSize() == UINT64_MAX))
+    error =
+        Status::FromErrorString("Invalid memory region from scripted process");
+
+  return error;
 }
 
 Status Process::GetMemoryRegions(lldb_private::MemoryRegionInfos &region_list) {