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 /binutils/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 'binutils/ChangeLog')
-rw-r--r-- | binutils/ChangeLog | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/binutils/ChangeLog b/binutils/ChangeLog index a577978..0d8bbc0 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,8 @@ +2018-04-05 H.J. Lu <hongjiu.lu@intel.com> + + PR gas/22318 + * configure: Regenerated. + 2018-04-05 Maciej W. Rozycki <macro@mips.com> * testsuite/binutils-all/strip-13mips64.s: Use `.dc.l' rather |