[Lldb-commits] [lldb] [lldb][windows] bound ReadProcessMemory (PR #200230)

[via lldb-commits]

================
@@ -614,44 +614,82 @@ static std::optional<std::string> GetFileNameByLoadAddress(HANDLE hProcess,
   return dos_path;
 }
 
+// Determine how many bytes can be read at `addr` in `hProcess` before crossing
+// out of the committed memory region containing it. Returns 0 if the address is
+// not within a committed region.
+static SIZE_T BytesReadableAt(HANDLE hProcess, LPCVOID addr) {
----------------
Nerixyz wrote:

There are some other variables also named `hProcess` in this file. I don't know what it should be changed to. Maybe `process` or `h_process`?

https://github.com/llvm/llvm-project/pull/200230