diff options
| author | H.J. Lu <hjl.tools@gmail.com> | 2018-04-05 15:31:41 -0700 |
|---|---|---|
| committer | H.J. Lu <hjl.tools@gmail.com> | 2018-04-05 15:31:53 -0700 |
| commit | 3cba8b6c93ab8c573ba4678255df11486de61c54 (patch) | |
| tree | c761a209fa3a5917064a32f1397fd8fab830fdba /gprof/ChangeLog | |
| parent | 552d4da4178cc5e8d567ee150462a28d51f08fc0 (diff) | |
| download | gdb-3cba8b6c93ab8c573ba4678255df11486de61c54.zip gdb-3cba8b6c93ab8c573ba4678255df11486de61c54.tar.gz gdb-3cba8b6c93ab8c573ba4678255df11486de61c54.tar.bz2 | |
Use dlsym to check if libdl is needed for plugin
config/plugins.m4 has
if test "$plugins" = "yes"; then
AC_SEARCH_LIBS([dlopen], [dl])
fi
Plugin uses dlsym, but libasan.so only intercepts dlopen, not dlsym:
[hjl@gnu-tools-1 binutils-text]$ nm -D /lib64/libasan.so.4| grep " dl"
0000000000038580 W dlclose
U dl_iterate_phdr
000000000004dc50 W dlopen
U dlsym
U dlvsym
[hjl@gnu-tools-1 binutils-text]$
Testing dlopen for libdl leads to false negative when -fsanitize=address
is used. It results in link failure:
../bfd/.libs/libbfd.a(plugin.o): undefined reference to symbol 'dlsym@@GLIBC_2.16'
dlsym should be used to check if libdl is needed for plugin.
bfd/
PR gas/22318
* configure: Regenerated.
binutils/
PR gas/22318
* configure: Regenerated.
gas/
PR gas/22318
* configure: Regenerated.
gprof/
PR gas/22318
* configure: Regenerated.
ld/
PR gas/22318
* configure: Regenerated.
Diffstat (limited to 'gprof/ChangeLog')
| -rw-r--r-- | gprof/ChangeLog | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/gprof/ChangeLog b/gprof/ChangeLog index bfe1f27..da15c98 100644 --- a/gprof/ChangeLog +++ b/gprof/ChangeLog @@ -1,3 +1,8 @@ +2018-04-05 H.J. Lu <hongjiu.lu@intel.com> + + PR gas/22318 + * configure: Regenerated. + 2018-03-01 Alan Modra <amodra@gmail.com> * configure.ac (ALL_LINGUAS): Add it, ro, ru, uk. Sort. |