jasonmolenda marked 2 inline comments as done.
jasonmolenda added inline comments.
================
Comment at: lldb/source/Plugins/Platform/MacOSX/PlatformDarwinKernel.cpp:924
+ addr_t input_addr) {
+ Status error;
+ WritableDataBufferSP header_data(new DataBufferHeap(512, 0));
----------------
JDevlieghere wrote:
> Looks like we're never checking this status below. Is there any chance that
> `ReadMemory` succeeds but the status is set to an error? If not, should we
> add an assert?
Good point. `Process::ReadMemoryFromInferior` returns the number of bytes
read, and a Status object. So if were were only able to read part of the mach
header object, this would not return an error condition. (and we'd have
uninitialized fields and could try to read too many load commands or something)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D133534/new/
https://reviews.llvm.org/D133534
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
