This patch is aimed at silencing gcc8 -Wstringop-truncation warnings.
Unfortunately adding __attribute__ ((__nonstring)) doesn't work in a
number of the places patched here, (see
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85643) so if you have
recent glibc headers installed you'll need to configure binutils with
--disable-werror to compile using gcc8 or gcc9.
include/
* ansidecl.h: Import from gcc.
* coff/internal.h (struct internal_scnhdr): Add ATTRIBUTE_NONSTRING
to s_name.
(struct internal_syment): Add ATTRIBUTE_NONSTRING to _n_name.
bfd/
* elf-linux-core.h (struct elf_external_linux_prpsinfo32_ugid32),
(struct elf_external_linux_prpsinfo32_ugid16),
(struct elf_external_linux_prpsinfo64_ugid32),
(struct elf_external_linux_prpsinfo64_ugid16): Add ATTRIBUTE_NONSTRING
to pr_fname and pr_psargs fields. Remove GCC diagnostic pragmas.
Move comment to..
* elf.c (elfcore_write_prpsinfo): ..here. Indent nested preprocessor
directives.
* elf32-arm.c (elf32_arm_nabi_write_core_note): Add ATTRIBUTE_NONSTRING
to data.
* elf32-ppc.c (ppc_elf_write_core_note): Likewise.
* elf32-s390.c (elf_s390_write_core_note): Likewise.
* elf64-s390.c (elf_s390_write_core_note): Likewise.
* elfxx-aarch64.c (_bfd_aarch64_elf_write_core_note): Likewise.
* elf64-x86-64.c (elf_x86_64_write_core_note): Add GCC diagnostic
pragmas.
* peXXigen.c (_bfd_XXi_swap_scnhdr_out): Use strnlen to avoid
false positive gcc-8 warning.
gas/
* config/obj-evax.c (shorten_identifier): Use memcpy in place
of strncpy.
* config/obj-macho.c (obj_mach_o_make_or_get_sect): Ensure
segname and sectname fields are NUL terminated.
+2018-05-04 Alan Modra <amodra@gmail.com>
+
+ * elf-linux-core.h (struct elf_external_linux_prpsinfo32_ugid32),
+ (struct elf_external_linux_prpsinfo32_ugid16),
+ (struct elf_external_linux_prpsinfo64_ugid32),
+ (struct elf_external_linux_prpsinfo64_ugid16): Add ATTRIBUTE_NONSTRING
+ to pr_fname and pr_psargs fields. Remove GCC diagnostic pragmas.
+ Move comment to..
+ * elf.c (elfcore_write_prpsinfo): ..here. Indent nested preprocessor
+ directives.
+ * elf32-arm.c (elf32_arm_nabi_write_core_note): Add ATTRIBUTE_NONSTRING
+ to data.
+ * elf32-ppc.c (ppc_elf_write_core_note): Likewise.
+ * elf32-s390.c (elf_s390_write_core_note): Likewise.
+ * elf64-s390.c (elf_s390_write_core_note): Likewise.
+ * elfxx-aarch64.c (_bfd_aarch64_elf_write_core_note): Likewise.
+ * elf64-x86-64.c (elf_x86_64_write_core_note): Add GCC diagnostic
+ pragmas.
+ * peXXigen.c (_bfd_XXi_swap_scnhdr_out): Use strnlen to avoid
+ false positive gcc-8 warning.
+
2018-05-04 Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
* elf32-ppc.c (_bfd_elf_ppc_merge_fp_attributes): Fix the order
case C_FCN:
if (obj_pe (input_bfd)
- && strcmp (isym.n_name, ".bf") != 0
+ && strncmp (isym.n_name, ".bf", sizeof isym.n_name) != 0
&& isym.n_scnum > 0)
{
/* For PE, .lf and .ef get their value left alone,
#ifndef ELF_LINUX_CORE_H
#define ELF_LINUX_CORE_H
-/* gcc-8 warns (*) on all the strncpy calls in this file about
- possible string truncation. The "truncation" is not a bug. We
- have an external representation of structs with fields that are not
- necessarily NULL terminated and corresponding internal
- representation fields that are one larger so that they can always
- be NULL terminated.
- gcc versions between 4.2 and 4.6 do not allow pragma control of
- diagnostics inside functions, giving a hard error if you try to use
- the finer control available with later versions.
- gcc prior to 4.2 warns about diagnostic push and pop.
- gcc-5, gcc-6 and gcc-7 warn that -Wstringop-truncation is unknown,
- unless you also add #pragma GCC diagnostic ignored "-Wpragma".
- (*) Depending on your system header files! */
-
-#if GCC_VERSION >= 8000
-#pragma GCC diagnostic push
-#pragma GCC diagnostic ignored "-Wstringop-truncation"
-#endif
-
/* External 32-bit structure for PRPSINFO. This structure is
ABI-defined, thus we choose to use char arrays here in order to
avoid dealing with different types in different architectures.
char pr_ppid[4];
char pr_pgrp[4];
char pr_sid[4];
- char pr_fname[16]; /* Filename of executable. */
- char pr_psargs[80]; /* Initial part of arg list. */
+ char pr_fname[16] ATTRIBUTE_NONSTRING; /* Filename of executable. */
+ char pr_psargs[80] ATTRIBUTE_NONSTRING; /* Initial part of arg list. */
};
/* Helper function to copy an elf_internal_linux_prpsinfo in host
char pr_ppid[4];
char pr_pgrp[4];
char pr_sid[4];
- char pr_fname[16]; /* Filename of executable. */
- char pr_psargs[80]; /* Initial part of arg list. */
+ char pr_fname[16] ATTRIBUTE_NONSTRING; /* Filename of executable. */
+ char pr_psargs[80] ATTRIBUTE_NONSTRING; /* Initial part of arg list. */
};
/* Helper function to copy an elf_internal_linux_prpsinfo in host
char pr_ppid[4];
char pr_pgrp[4];
char pr_sid[4];
- char pr_fname[16]; /* Filename of executable. */
- char pr_psargs[80]; /* Initial part of arg list. */
+ char pr_fname[16] ATTRIBUTE_NONSTRING; /* Filename of executable. */
+ char pr_psargs[80] ATTRIBUTE_NONSTRING; /* Initial part of arg list. */
};
/* Helper function to copy an elf_internal_linux_prpsinfo in host
char pr_ppid[4];
char pr_pgrp[4];
char pr_sid[4];
- char pr_fname[16]; /* Filename of executable. */
- char pr_psargs[80]; /* Initial part of arg list. */
+ char pr_fname[16] ATTRIBUTE_NONSTRING; /* Filename of executable. */
+ char pr_psargs[80] ATTRIBUTE_NONSTRING; /* Initial part of arg list. */
};
/* Helper function to copy an elf_internal_linux_prpsinfo in host
strncpy (to->pr_psargs, from->pr_psargs, sizeof (to->pr_psargs));
}
-#if GCC_VERSION >= 8000
-#pragma GCC diagnostic pop
-#endif
-
#endif
return buf;
}
+/* gcc-8 warns (*) on all the strncpy calls in this function about
+ possible string truncation. The "truncation" is not a bug. We
+ have an external representation of structs with fields that are not
+ necessarily NULL terminated and corresponding internal
+ representation fields that are one larger so that they can always
+ be NULL terminated.
+ gcc versions between 4.2 and 4.6 do not allow pragma control of
+ diagnostics inside functions, giving a hard error if you try to use
+ the finer control available with later versions.
+ gcc prior to 4.2 warns about diagnostic push and pop.
+ gcc-5, gcc-6 and gcc-7 warn that -Wstringop-truncation is unknown,
+ unless you also add #pragma GCC diagnostic ignored "-Wpragma".
+ (*) Depending on your system header files! */
#if GCC_VERSION >= 8000
-#pragma GCC diagnostic push
-#pragma GCC diagnostic ignored "-Wstringop-truncation"
+# pragma GCC diagnostic push
+# pragma GCC diagnostic ignored "-Wstringop-truncation"
#endif
char *
elfcore_write_prpsinfo (bfd *abfd,
}
#if defined (HAVE_PRPSINFO_T) || defined (HAVE_PSINFO_T)
-#if defined (HAVE_PRPSINFO32_T) || defined (HAVE_PSINFO32_T)
+# if defined (HAVE_PRPSINFO32_T) || defined (HAVE_PSINFO32_T)
if (bed->s->elfclass == ELFCLASS32)
{
-#if defined (HAVE_PSINFO32_T)
+# if defined (HAVE_PSINFO32_T)
psinfo32_t data;
int note_type = NT_PSINFO;
-#else
+# else
prpsinfo32_t data;
int note_type = NT_PRPSINFO;
-#endif
+# endif
memset (&data, 0, sizeof (data));
strncpy (data.pr_fname, fname, sizeof (data.pr_fname));
"CORE", note_type, &data, sizeof (data));
}
else
-#endif
+# endif
{
-#if defined (HAVE_PSINFO_T)
+# if defined (HAVE_PSINFO_T)
psinfo_t data;
int note_type = NT_PSINFO;
-#else
+# else
prpsinfo_t data;
int note_type = NT_PRPSINFO;
-#endif
+# endif
memset (&data, 0, sizeof (data));
strncpy (data.pr_fname, fname, sizeof (data.pr_fname));
return NULL;
}
#if GCC_VERSION >= 8000
-#pragma GCC diagnostic pop
+# pragma GCC diagnostic pop
#endif
char *
case NT_PRPSINFO:
{
- char data[124];
+ char data[124] ATTRIBUTE_NONSTRING;
va_list ap;
va_start (ap, note_type);
case NT_PRPSINFO:
{
- char data[128];
+ char data[128] ATTRIBUTE_NONSTRING;
va_list ap;
va_start (ap, note_type);
case NT_PRPSINFO:
{
- char data[124] = { 0 };
+ char data[124] ATTRIBUTE_NONSTRING = { 0 };
const char *fname, *psargs;
va_start (ap, note_type);
case NT_PRPSINFO:
{
- char data[136] = { 0 };
+ char data[136] ATTRIBUTE_NONSTRING = { 0 };
const char *fname, *psargs;
va_start (ap, note_type);
}
#ifdef CORE_HEADER
+# if GCC_VERSION >= 8000
+# pragma GCC diagnostic push
+# pragma GCC diagnostic ignored "-Wstringop-truncation"
+# endif
static char *
elf_x86_64_write_core_note (bfd *abfd, char *buf, int *bufsiz,
int note_type, ...)
}
/* NOTREACHED */
}
+# if GCC_VERSION >= 8000
+# pragma GCC diagnostic pop
+# endif
#endif
\f
/* Functions for the x86-64 ELF linker. */
case NT_PRPSINFO:
{
- char data[136];
+ char data[136] ATTRIBUTE_NONSTRING;
va_list ap;
va_start (ap, note_type);
by ld --omagic, or by obcopy --writable-text. */
for (p = known_sections; p->section_name; p++)
- if (strcmp (scnhdr_int->s_name, p->section_name) == 0)
+ if (strncmp (scnhdr_int->s_name, p->section_name,
+ sizeof scnhdr_int->s_name) == 0)
{
- if (strcmp (scnhdr_int->s_name, ".text")
+ if (strncmp (scnhdr_int->s_name, ".text", sizeof scnhdr_int->s_name)
|| (bfd_get_file_flags (abfd) & WP_TEXT))
scnhdr_int->s_flags &= ~IMAGE_SCN_MEM_WRITE;
scnhdr_int->s_flags |= p->must_have;
if (coff_data (abfd)->link_info
&& ! bfd_link_relocatable (coff_data (abfd)->link_info)
&& ! bfd_link_pic (coff_data (abfd)->link_info)
- && strcmp (scnhdr_int->s_name, ".text") == 0)
+ && strncmp (scnhdr_int->s_name, ".text", sizeof scnhdr_int->s_name) == 0)
{
/* By inference from looking at MS output, the 32 bit field
which is the combination of the number_of_relocs and
+2018-05-04 Alan Modra <amodra@gmail.com>
+
+ * config/obj-evax.c (shorten_identifier): Use memcpy in place
+ of strncpy.
+ * config/obj-macho.c (obj_mach_o_make_or_get_sect): Ensure
+ segname and sectname fields are NUL terminated.
+
2018-05-01 Nick Clifton <nickc@redhat.com>
* po/es.po: Updated Spanish translation.
newname [MAX_LABEL_LENGTH] = 0;
/* Now append the suffix of the original identifier, if any. */
if (suffix_length)
- strncpy (newname + MAX_LABEL_LENGTH - suffix_length,
- name + len - suffix_length,
- suffix_length);
- strncpy (newname + final_len, "_h", 2);
- strncpy (newname + final_len + 2 , crc_chars, 5);
- strncpy (newname + final_len + 2 + 5, encode_16 (len), 3);
- strncpy (newname + final_len + 2 + 5 + 3, encode_16 (sum), 3);
+ strncpy (newname + MAX_LABEL_LENGTH - suffix_length,
+ name + len - suffix_length,
+ suffix_length);
+ memcpy (newname + final_len, "_h", 2);
+ memcpy (newname + final_len + 2 , crc_chars, 5);
+ memcpy (newname + final_len + 2 + 5, encode_16 (len), 3);
+ memcpy (newname + final_len + 2 + 5 + 3, encode_16 (sum), 3);
if (!is_truncated_identifier (newname))
abort ();
return newname;
bfd_section_name (stdoutput, sec),
bfd_errmsg (bfd_get_error ()));
- strncpy (msect->segname, segname, sizeof (msect->segname));
- strncpy (msect->sectname, sectname, sizeof (msect->sectname));
+ strncpy (msect->segname, segname, BFD_MACH_O_SEGNAME_SIZE);
+ msect->segname[BFD_MACH_O_SEGNAME_SIZE] = 0;
+ strncpy (msect->sectname, sectname, BFD_MACH_O_SECTNAME_SIZE);
+ msect->sectname[BFD_MACH_O_SECTNAME_SIZE] = 0;
msect->align = secalign;
msect->flags = sectype | secattr;
+2018-05-04 Alan Modra <amodra@gmail.com>
+
+ * ansidecl.h: Import from gcc.
+ * coff/internal.h (struct internal_scnhdr): Add ATTRIBUTE_NONSTRING
+ to s_name.
+ (struct internal_syment): Add ATTRIBUTE_NONSTRING to _n_name.
+
2018-04-30 Francois H. Theron <francois.theron@netronome.com>
* dis-asm.h: Added print_nfp_disassembler_options prototype.
# endif /* GNUC >= 4.9 */
#endif /* ATTRIBUTE_NO_SANITIZE_UNDEFINED */
+/* Attribute 'nonstring' was valid as of gcc 8. */
+#ifndef ATTRIBUTE_NONSTRING
+# if GCC_VERSION >= 8000
+# define ATTRIBUTE_NONSTRING __attribute__ ((__nonstring__))
+# else
+# define ATTRIBUTE_NONSTRING
+# endif
+#endif
+
/* We use __extension__ in some places to suppress -pedantic warnings
about GCC extensions. This feature didn't work properly before
gcc 2.8. */
struct internal_scnhdr
{
- char s_name[SCNNMLEN]; /* section name */
+ char s_name[SCNNMLEN] ATTRIBUTE_NONSTRING; /* section name */
/* Physical address, aliased s_nlib.
In the pei format, this field is the virtual section size
{
union
{
- char _n_name[SYMNMLEN]; /* old COFF version */
+ char _n_name[SYMNMLEN] ATTRIBUTE_NONSTRING; /* old COFF version */
struct
{
bfd_hostptr_t _n_zeroes; /* new == 0 */
projects / binutils-gdb.git / commitdiff