[all-commits] [llvm/llvm-project] a59bb2: [ifs] Added missing DT_STRSZ to the .dynamic section

Zero Omega via All-commits all-commits at lists.llvm.org
Sun Feb 13 21:06:10 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a59bb21bf85ebf5fda18f331ea6ff433a4ad73aa
      https://github.com/llvm/llvm-project/commit/a59bb21bf85ebf5fda18f331ea6ff433a4ad73aa
  Author: Haowei Wu <haowei at google.com>
  Date:   2022-02-13 (Sun, 13 Feb 2022)

  Changed paths:
    M llvm/lib/InterfaceStub/ELFObjHandler.cpp
    M llvm/test/tools/llvm-ifs/write-stub.test

  Log Message:
  -----------
  [ifs] Added missing DT_STRSZ to the .dynamic section

This patch adds DT_STRSZ into the dynamic section, which was absent
previously. This was a bug and caused failures in other tools.

Differential Revision: https://reviews.llvm.org/D117058


  Commit: 15dfe03022caf38d173bd2d950711968c6c31d60
      https://github.com/llvm/llvm-project/commit/15dfe03022caf38d173bd2d950711968c6c31d60
  Author: Haowei Wu <haowei at google.com>
  Date:   2022-02-13 (Sun, 13 Feb 2022)

  Changed paths:
    M llvm/lib/InterfaceStub/ELFObjHandler.cpp
    M llvm/test/tools/llvm-ifs/binary-read-bad-vaddr.test
    A llvm/test/tools/llvm-ifs/ifs-elf-conversion.test

  Log Message:
  -----------
  [ifs] Allow llvm-ifs to generate text stub from elf stub

ELF stubs generated from llvm-ifs lacks program headers, which prevents
llvm-ifs from parsing them properly as program headers are required by
llvm's own ELF libraries. This patch adds a few workaround bypass this
limitation.

Differential Revision: https://reviews.llvm.org/D116769


Compare: https://github.com/llvm/llvm-project/compare/7f51a9e2730e...15dfe03022ca


More information about the All-commits mailing list