[Lldb-commits] [PATCH] D57037: BreakpadRecords: Address post-commit feedback

Greg Clayton via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Jan 22 08:59:58 PST 2019

clayborg added inline comments.

Comment at: source/Plugins/ObjectFile/Breakpad/BreakpadRecords.cpp:87-97
   // The textual module id encoding should be between 33 and 40 bytes long,
   // depending on the size of the age field, which is of variable length.
   // The first three chunks of the id are encoded in big endian, so we need to
   // byte-swap those.
-  if (str.size() < 33 || str.size() > 40)
+  if (str.size() <= hex_digits<data_t::uuid_t>() ||
+      str.size() > hex_digits<data_t>())
     return UUID();
This is OK as long as the UUIDs for ELF don't fall into this category. I am able to match up UUIDs for ELF just fine for breakpad files for Android.



More information about the lldb-commits mailing list