[binutils-gdb] libctf: add missing header in BFD ELF check
Nick Alcock
nix@sourceware.org
Wed Feb 10 15:38:59 GMT 2021
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=758f590744b1cf8d1049fca3223d1817242cacb8
commit 758f590744b1cf8d1049fca3223d1817242cacb8
Author: Nick Alcock <nick.alcock@oracle.com>
Date: Wed Feb 3 14:09:02 2021 +0000
libctf: add missing header in BFD ELF check
Without this, GCC warns:
In file included from conftest.c:36:
../../libctf/../bfd/elf-bfd.h: In function 'bfd_section_is_ctf':
../../libctf/../bfd/elf-bfd.h:3089:10: warning: implicit declaration of function 'strncmp' [-Wimplicit-function-declaration]
3089 | return strncmp (name, ".ctf", 4) == 0 && (name[4] == 0 || name[4] == '.');
| ^~~~~~~
../../libctf/../bfd/elf-bfd.h:3089:33: warning: 'strncmp' argument 3 type is 'int' where 'long unsigned int' is expected in a call to built-in function declared without prototype [-Wbuiltin-declaration-mismatch]
3089 | return strncmp (name, ".ctf", 4) == 0 && (name[4] == 0 || name[4] == '.');
| ^
<built-in>: note: built-in 'strncmp' declared here
These warnings do not currently throw off the result of the configure
check, but it's better to squash them anyway.
libctf/ChangeLog
2021-02-03 Nick Alcock <nick.alcock@oracle.com>
* configure.ac (ac_cv_libctf_bfd_elf): Include string.h.
* configure: Regenerated.
Diff:
---
libctf/ChangeLog | 5 +++++
libctf/configure | 1 +
libctf/configure.ac | 1 +
3 files changed, 7 insertions(+)
diff --git a/libctf/ChangeLog b/libctf/ChangeLog
index 135add5a6c2..cf529038d70 100644
--- a/libctf/ChangeLog
+++ b/libctf/ChangeLog
@@ -1,3 +1,8 @@
+2021-02-03 Nick Alcock <nick.alcock@oracle.com>
+
+ * configure.ac (ac_cv_libctf_bfd_elf): Include string.h.
+ * configure: Regenerated.
+
2021-02-03 Nick Alcock <nick.alcock@oracle.com>
* configure.ac (EXPECT): Check for, in order to define...
diff --git a/libctf/configure b/libctf/configure
index 902eadd5d7a..d5b78611574 100755
--- a/libctf/configure
+++ b/libctf/configure
@@ -12875,6 +12875,7 @@ else
cat confdefs.h - <<_ACEOF >conftest.$ac_ext
/* end confdefs.h. */
#include <stdlib.h>
+ #include <string.h>
#include "bfd.h"
#include "elf-bfd.h"
int
diff --git a/libctf/configure.ac b/libctf/configure.ac
index c6b80442e07..90e3c437d60 100644
--- a/libctf/configure.ac
+++ b/libctf/configure.ac
@@ -88,6 +88,7 @@ intl=`echo $LIBINTL | sed 's,${top_builddir}/,,g'`
LIBS="-lbfd -liberty -lz $intl $LIBS"
AC_CACHE_CHECK([for ELF support in BFD], ac_cv_libctf_bfd_elf,
[AC_TRY_LINK([#include <stdlib.h>
+ #include <string.h>
#include "bfd.h"
#include "elf-bfd.h"],
[(void) bfd_section_from_elf_index (NULL, 0);
More information about the Binutils-cvs
mailing list