[lldb/crashlog] Skip images with empty path and 0 UUID from loading

[mib]

Summary

This patch skips images with an empty path or a 0 UUID from loading as a
SymbolFileJSON.

rdar://112107986

Signed-off-by: Med Ismail Bennani <[e-mail]>

[bulbazord]

lldb/examples/python/symbolication.py

413–418

It looks like name is only used below on line 435 to create obj_file, so you could move that computation down there and check the validity of self.path instead. Does that work? Or am I missing something?

Also, it seems like you might want to do this chunk of work at the beginning of the function to make sure you have a valid image before you try to process it? Is there an edge case where we'd still want to process a module with a UUID of 0 and no path?

[mib]

Address @bulbazord comments.

[bulbazord]

LGTM

[JDevlieghere]

Test?

lldb/examples/python/symbolication.py

def add_module(self, target, obj_dir=None):
             if not self.module and self.resolve:
                 self.locate_module_and_debug_symbols()
                 if not self.unavailable:
                     resolved_path = self.get_resolved_path()
                     self.module = target.AddModule(
                         resolved_path, None, uuid_str, self.symfile
                     )
             if not self.module and self.section_infos:
                 name = os.path.basename(self.path)
+                if not name:
+                    if self.uuid == uuid.UUID(int=0):
+                        return None
+                    else:
+                        return "error: invalid image path with valid UUID (%s) " % uuid_str
                 if obj_dir and os.path.isdir(obj_dir):
                     data = {
                         "triple": target.triple,
                         "uuid": uuid_str,
                         "type": "sharedlibrary",
                         "sections": list(),
                         "symbols": list(),
                     }