[Lldb-commits] [lldb] [lldb] Change DynamicLoaderDarwin::Segment::name type (PR #210797)

[Jason Molenda via lldb-commits]

================
@@ -881,8 +881,8 @@ uint32_t DynamicLoaderMacOSXDYLD::ParseLoadCommands(const DataExtractor &data,
       } break;
 
       case llvm::MachO::LC_SEGMENT_64: {
-        segment.name.SetTrimmedCStringWithLength(
-            (const char *)data.GetData(&offset, 16), 16);
+        data.CopyData(offset, 16, segment.name);
----------------
jasonmolenda wrote:

You need to seg `segment.name[16] = '\0'` on both of these to ensure it is nul-terminated.

https://github.com/llvm/llvm-project/pull/210797