[all-commits] [llvm/llvm-project] 84a1bc: [test/Object][llvm-objdump] - llvm-objdump: don't ...

Georgii Rymar via All-commits all-commits at lists.llvm.org
Tue Jul 14 04:58:09 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 84a1bc7f2c0c7bd5f18a4ecaf91e27644aa94190
      https://github.com/llvm/llvm-project/commit/84a1bc7f2c0c7bd5f18a4ecaf91e27644aa94190
  Author: Georgii Rymar <grimar at accesssoftek.com>
  Date:   2020-07-14 (Tue, 14 Jul 2020)

  Changed paths:
    R llvm/test/Object/Inputs/invalid-phdr.elf
    R llvm/test/Object/elf-invalid-phdr.test
    M llvm/test/Object/invalid.test
    A llvm/test/tools/llvm-objdump/ELF/invalid-phdr.test
    M llvm/tools/llvm-objdump/ELFDump.cpp

  Log Message:
  -----------
  [test/Object][llvm-objdump] - llvm-objdump: don't abort() when the e_phoff field is invalid and refine testing.

llvm-objdump currently calls report_fatal_error() when the e_phoff field is invalid.

This is tested by elf-invalid-phdr.test which has the following issues:
1) It uses a precompiled object.
2) it could be a part of invalid.test.
3) It tests the Object lib, but we have no separate test for llvm-objdump.

This patch addresses issues mentioned.

Differential revision: https://reviews.llvm.org/D83559




More information about the All-commits mailing list