[DynInst_API:] [dyninst/dyninst] ad753b: Do not put newly added functions to .dynsym, which...


Date: Fri, 08 Feb 2019 13:15:12 -0800
From: Xiaozhu Meng <noreply@xxxxxxxxxx>
Subject: [DynInst_API:] [dyninst/dyninst] ad753b: Do not put newly added functions to .dynsym, which...
  Branch: refs/heads/master
  Home:   https://github.com/dyninst/dyninst
  Commit: ad753b233f77bc38fb8130671fc4acdfa57d031e
      https://github.com/dyninst/dyninst/commit/ad753b233f77bc38fb8130671fc4acdfa57d031e
  Author: Xiaozhu Meng <xmeng@xxxxxxxxxxx>
  Date:   2019-02-06 (Wed, 06 Feb 2019)

  Changed paths:
    M symtabAPI/src/Symtab-edit.C

  Log Message:
  -----------
  Do not put newly added functions to .dynsym, which may cause the load
to behave abnormally due to duplicated symbols.

Note that adding symbols to .dynsym can be a legit use case, where
we expose hidden functions in shared libraries. But this use case
needs more development to support.


  Commit: 8ff6621c65b04d100ff164e771482f97bef8cd02
      https://github.com/dyninst/dyninst/commit/8ff6621c65b04d100ff164e771482f97bef8cd02
  Author: Xiaozhu Meng <mxz297@xxxxxxxxx>
  Date:   2019-02-08 (Fri, 08 Feb 2019)

  Changed paths:
    M symtabAPI/src/Symtab-edit.C

  Log Message:
  -----------
  Merge pull request #548 from dyninst/dynsym_fix

Do not put newly added functions to .dynsym, which may cause the load


Compare: https://github.com/dyninst/dyninst/compare/d6f7ae96cc9d...8ff6621c65b0
[← Prev in Thread] Current Thread [Next in Thread→]