[llvm] [llvm-objdump][ELF]Fix crash when reading strings from .dynstr (PR #125679)

[James Henderson via llvm-commits]

================
@@ -470,3 +470,50 @@ Sections:
        Value: 0x1
      - Tag:   DT_NULL
        Value: 0x0
+
+# RUN: yaml2obj --docnum=5 %s -o %t5
+# RUN: llvm-objdump -p %t5 2>&1 | FileCheck %s --strict-whitespace -DFILE=%t5 --check-prefix=WARN
+
+# WARN: Program Header:
+# WARN:    LOAD off    0x00000000000000b0 vaddr 0x0000000000001000 paddr 0x0000000000001000 align 2**0
+# WARN:         filesz 0x0000000000000050 memsz 0x0000000000000050 flags ---
+# WARN: DYNAMIC off    0x00000000000000c0 vaddr 0x000000000000101d paddr 0x000000000000101d align 2**0
+# WARN:         filesz 0x0000000000000040 memsz 0x0000000000000040 flags ---
+# WARN: Dynamic Section:
+# WARN: warning: '[[FILE]]': invalid string table offset
+# WARN:  NEEDED 0x0000000000000011
+# WARN:  STRTAB 0x0000000000001000
+# WARN:  STRSZ  0x0000000000000010
+
+--- !ELF
+FileHeader:
+  Class:   ELFCLASS64
+  Data:    ELFDATA2LSB
+  Type:    ET_EXEC
+  Machine: EM_X86_64
+Sections:
+  - Name:    .dynstr
+    Type:    SHT_STRTAB
+    Address: 0x1000
+    Size:    0x10
+    Content: "004400550066007700"
----------------
jh7370 wrote:

I don't think you need both `Size` and `Content`. Just stick with `Size`, since we don't need real content to display.

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