[llvm] r345503 - [llvm-objdump] Don't crash when using `-a` on non-archives
James Henderson via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 29 07:17:08 PDT 2018
Author: jhenderson
Date: Mon Oct 29 07:17:08 2018
New Revision: 345503
URL: http://llvm.org/viewvc/llvm-project?rev=345503&view=rev
Log:
[llvm-objdump] Don't crash when using `-a` on non-archives
This fixes PR39402. The crash was caused when dereferencing nullptr in
DumpObject and printArchiveChild.
Reviewed By: jhenderson
Differential Revision: https://reviews.llvm.org/D53690
Patch by Xing GUO
Added:
llvm/trunk/test/tools/llvm-objdump/non-archive-object.test
Modified:
llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp
Added: llvm/trunk/test/tools/llvm-objdump/non-archive-object.test
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-objdump/non-archive-object.test?rev=345503&view=auto
==============================================================================
--- llvm/trunk/test/tools/llvm-objdump/non-archive-object.test (added)
+++ llvm/trunk/test/tools/llvm-objdump/non-archive-object.test Mon Oct 29 07:17:08 2018
@@ -0,0 +1,25 @@
+# RUN: yaml2obj %s > %t
+# RUN: llvm-objdump -a %t | FileCheck %s
+
+# If this test has not crashed, then this test passed.
+# CHECK: file format ELF64-x86-64
+
+!ELF
+FileHeader:
+ Class: ELFCLASS64
+ Data: ELFDATA2LSB
+ Type: ET_EXEC
+ Machine: EM_X86_64
+Sections:
+ - Name: .bss
+ Type: SHT_NOBITS
+ Flags: [ SHF_ALLOC ]
+ AddressAlign: 0x0000000000000010
+ Size: 64
+ - Name: .text
+ Type: SHT_PROGBITS
+ Flags: [ SHF_ALLOC, SHF_EXECINSTR ]
+ AddressAlign: 0x0000000000000010
+ Content: "01234567"
+ Size: 4
+
Modified: llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp?rev=345503&r1=345502&r2=345503&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp (original)
+++ llvm/trunk/tools/llvm-objdump/llvm-objdump.cpp Mon Oct 29 07:17:08 2018
@@ -2309,8 +2309,8 @@ static void DumpObject(ObjectFile *o, co
outs() << ":\tfile format " << o->getFileFormatName() << "\n\n";
}
- if (ArchiveHeaders && !MachOOpt)
- printArchiveChild(a->getFileName(), *c);
+ if (ArchiveHeaders && !MachOOpt && c)
+ printArchiveChild(ArchiveName, *c);
if (Disassemble)
DisassembleObject(o, Relocations);
if (Relocations && !Disassemble)
@@ -2363,8 +2363,8 @@ static void DumpObject(const COFFImportF
<< ":\tfile format COFF-import-file"
<< "\n\n";
- if (ArchiveHeaders && !MachOOpt)
- printArchiveChild(A->getFileName(), *C);
+ if (ArchiveHeaders && !MachOOpt && C)
+ printArchiveChild(ArchiveName, *C);
if (SymbolTable)
printCOFFSymbolTable(I);
}
More information about the llvm-commits
mailing list