[DynInst_API:] [dyninst/dyninst] 9457fc: Fix for test1_30:


Date: Thu, 06 Sep 2018 15:15:44 -0700
From: Sasha Nicolas <sasha@xxxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] 9457fc: Fix for test1_30:
  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: 9457fc40e882d9da4d7b6e21762ac24141f2bfe9
      https://github.com/dyninst/dyninst/commit/9457fc40e882d9da4d7b6e21762ac24141f2bfe9
  Author: Sasha Nicolas <sasha@xxxxxxxxxxx>
  Date:   2018-09-05 (Wed, 05 Sep 2018)

  Changed paths:
    M cmake/packages.cmake
    M patchAPI/CMakeLists.txt
    M stackwalk/CMakeLists.txt
    M symtabAPI/CMakeLists.txt
    M symtabAPI/h/LineInformation.h
    M symtabAPI/h/StringTable.h
    M symtabAPI/src/Function.C
    M symtabAPI/src/LineInformation.C
    M symtabAPI/src/Object-elf.C
    M symtabAPI/src/dwarfWalker.C

  Log Message:
  -----------
  Fix for test1_30:
this test would show PASSED but in reality the return value of the
functions getAddressRanges either in BPatch_image or BPatch_module was
false, meaning the ranges weren't being found correctly.
It turns out that the comparison between filenames was wrong.
For this fix, StringTableEntry needed to be changed in other to store the filename
without the path, and thus allow StringTable boost::multi_index_container
to be indexed by "filename" and not only by "/path/filename".


  Commit: 7a6870835880edd49a96abe3344f7f27911ccd5f
      https://github.com/dyninst/dyninst/commit/7a6870835880edd49a96abe3344f7f27911ccd5f
  Author: Sasha Nicolas <sasha@xxxxxxxxxxx>
  Date:   2018-09-06 (Thu, 06 Sep 2018)

  Changed paths:
    M symtabAPI/src/Object-elf.C

  Log Message:
  -----------
  Setting the default value of pointers encoding
in a Frame Description Entry (FDE) to be DW_EH_PE_absptr.


Compare: https://github.com/dyninst/dyninst/compare/7ead1e045f1e...7a6870835880
      **NOTE:** This service has been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.
[← Prev in Thread] Current Thread [Next in Thread→]
  • [DynInst_API:] [dyninst/dyninst] 9457fc: Fix for test1_30:, Sasha Nicolas <=