[Lldb-commits] [PATCH] D23545: Minidump parsing
Pavel Labath via lldb-commits
lldb-commits at lists.llvm.org
Fri Aug 19 06:04:00 PDT 2016
labath added a comment.
Looks good as far as I am concerned. You might want to add a test or two that makes sure we handle invalid data reasonably. E.g., load only the first X kb of the minidump (so that the thread list is not present in the loaded block), and make sure getting a thread list fails (and does not crash).
Comment at: source/Plugins/Process/minidump/MinidumpParser.cpp:40
@@ +39,3 @@
+ // check if there is enough data for the parsing of the directory list
+ if ((directory_list_offset + sizeof(MinidumpDirectory) * m_header->streams_count) > m_data_sp->GetByteSize())
Please put this before the construction of the ArrayRef - so we don't create a dangling object for no reason.
More information about the lldb-commits