[Lldb-commits] [PATCH] D86261: Add hashing of the .text section to ProcessMinidump.
Greg Clayton via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Thu Aug 20 14:12:47 PDT 2020
clayborg added inline comments.
Comment at: lldb/source/Plugins/Process/minidump/ProcessMinidump.cpp:174
+ // end of the .text section if the .text section is less than a page size in
+ // length.
+ const uint8_t *ptr = text_data.GetDataStart();
> I will try and make sure the data is there. The main issue is I don't think you can ask for more bytes than a section has, I believe it will cap the data. But I will check into this.
Never mind, you are reading from the object file's data directly (not the section contents) so this should work just fine.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the lldb-commits