Don't include the Age in the UUID for CvRecordPdb70 UUID records in minidump files

[clayborg]

Summary

The CvRecordPdb70 structure looks like:

struct CvRecordPdb70 {
  uint8_t Uuid[16];
  llvm::support::ulittle32_t Age;
  // char PDBFileName[];
};

We are currently including the "Age" in the UUID which seems wrong. I am proposing this fix to avoid including the age in the UUID since it is wrong for Apple targets. We want the UUID of a module to match the UUID that would be found in executable files. I can modify this patch to only do this for Apple vendors in the target triple if needed, but it seemed like this would be the functionality that people would expect on all targets, so I will start with this patch and see what people think.

[zturner]

For PE/COFF files, the Age is also in the executable and Guid+Age actually
constitute a 20-byte UUID. Is this not the case on Apple? What object file
format are you dealing with?

[lemo]

I'm curious too: where did the PDB70 age create matching problems?

On a related note, I just noticed that ObjectFilePECOFF::GetUUID() doesn't
have a real implementation (just returns false). How do we extract module
UUID for PE/COFF files?

[clayborg]

In D51442#1217829, @zturner wrote:

For PE/COFF files, the Age is also in the executable and Guid+Age actually
constitute a 20-byte UUID. Is this not the case on Apple? What object file
format are you dealing with?

Breakpad files that contain darwin crashes have the age set to zero, and then this hoses us the ability to match UUIDs for Apple stuff. Sounds like I need to make this apple specific

[clayborg]

Make 16 byte UUIDs Apple specific.

[clayborg]

In D51442#1217894, @lemo wrote:

I'm curious too: where did the PDB70 age create matching problems?

For breakpad ARM and ARM64 minidumps that are for Apple vendor triples.

On a related note, I just noticed that ObjectFilePECOFF::GetUUID() doesn't
have a real implementation (just returns false). How do we extract module
UUID for PE/COFF files?

Not sure on that, nor what the PDB70 UUID actually is.

[lemo]

Looks good (with one inline request for a comment)

source/Plugins/Process/minidump/MinidumpParser.cpp

86

please add a comment describing the situation (breakpad producing debug ids which don't match the binaries)

source/Plugins/Process/minidump/MinidumpParser.cpp

UUID MinidumpParser::GetModuleUUID(const MinidumpModule *module) {
   const CvSignature cv_signature =
       static_cast<CvSignature>(static_cast<const uint32_t>(*signature));
 
   if (cv_signature == CvSignature::Pdb70) {
     // PDB70 record
     const CvRecordPdb70 *pdb70_uuid = nullptr;
     Status error = consumeObject(cv_record, pdb70_uuid);
     if (!error.Fail())
-      return UUID::fromData(pdb70_uuid, sizeof(*pdb70_uuid));
+      return UUID::fromData(pdb70_uuid->Uuid, sizeof(pdb70_uuid->Uuid));
   } else if (cv_signature == CvSignature::ElfBuildId)
     return UUID::fromData(cv_record);
 
   return UUID();
 }
 
 llvm::ArrayRef<MinidumpThread> MinidumpParser::GetThreads() {
   llvm::ArrayRef<uint8_t> data = GetStream(MinidumpStreamType::ThreadList);
 
   if (data.size() == 0)