[Lldb-commits] [PATCH] D59433: Fix UUID decoding from minidump files.
Greg Clayton via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Mar 20 08:11:52 PDT 2019
clayborg marked an inline comment as done.
clayborg added inline comments.
Comment at: packages/Python/lldbsuite/test/functionalities/postmortem/minidump-new/TestMiniDumpNew.py:533
+ self.assertEqual(2, len(modules))
+ self.verify_module(modules, "/tmp/a", None)
+ self.verify_module(modules, "/tmp/b", None)
> labath wrote:
> > Am I right in thinking that if I have an object file called `/tmp/a` on my system, then lldb will pick it up (and it's build-id), causing this test to fail ? If that's the case then it would be better to use some path which is less likely to exist.
> I second this. https://docs.python.org/3/library/tempfile.html provides ways to create such temporary names.
These have strict UUID values that must match and the value is hard coded into the minidump files. So even if there is such a file, the UUID won't match.
CHANGES SINCE LAST ACTION
More information about the lldb-commits