[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