From 49c76560f31772bb6518c7e94549e44a19c25a7e Mon Sep 17 00:00:00 2001 From: Rui Ueyama Date: Fri, 28 Feb 2014 05:21:29 +0000 Subject: [PATCH] llvm-objdump: Fix crash bug with printing unwind info on stripped file. The current COFF unwind printer tries to print SEH handler function names, assuming that it can always find function names in string table. It crashes if file being read has no symbol table (i.e. executable). With this patch, llvm-objdump prints SEH handler's RVA if there's no symbol table entry for that RVA. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@202466 91177308-0d34-0410-b5e6-96231b3b80d8 --- tools/llvm-objdump/COFFDump.cpp | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/tools/llvm-objdump/COFFDump.cpp b/tools/llvm-objdump/COFFDump.cpp index c4bd81adedc..0b7f64d39e6 100644 --- a/tools/llvm-objdump/COFFDump.cpp +++ b/tools/llvm-objdump/COFFDump.cpp @@ -182,10 +182,10 @@ static error_code resolveSymbol(const std::vector &Rels, return EC; if (Ofs == Offset) { Sym = *I->getSymbol(); - break; + return object_error::success; } } - return object_error::success; + return object_error::parse_failed; } // Given a vector of relocations for a section and an offset into this section @@ -225,11 +225,13 @@ static void printCOFFSymbolAddress(llvm::raw_ostream &Out, const std::vector &Rels, uint64_t Offset, uint32_t Disp) { StringRef Sym; - if (error(resolveSymbolName(Rels, Offset, Sym))) - return; - Out << Sym; - if (Disp > 0) - Out << format(" + 0x%04x", Disp); + if (!resolveSymbolName(Rels, Offset, Sym)) { + Out << Sym; + if (Disp > 0) + Out << format(" + 0x%04x", Disp); + } else { + Out << format("0x%04x", Disp); + } } static void -- 2.34.1