From: H.J. Lu Date: Tue, 2 May 2017 19:16:26 +0000 (-0700) Subject: Cast relcount to unsigned long when comparing with sec->reloc_count X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=2ecf0cc317d065cfeb960c61688897351521bce0;p=binutils-gdb.git Cast relcount to unsigned long when comparing with sec->reloc_count The type of relcount is long and the type of sec->reloc_count is unsigned int. On 32-bit hosts, GCC issues an error: objcopy.c:2144:20: error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] if (relcount > sec->reloc_count) Cast relcount to unsigned long to silence GCC. * objcopy.c (merge_gnu_build_notes): Cast relcount to unsigned long when comparing with sec->reloc_count. --- diff --git a/binutils/ChangeLog b/binutils/ChangeLog index d13dbb637d4..b3a539ab49e 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,8 @@ +2017-05-02 H.J. Lu + + * objcopy.c (merge_gnu_build_notes): Cast relcount to unsigned + long when comparing with sec->reloc_count. + 2017-05-02 Nick Clifton * objcopy.c (merge_gnu_build_notes): Disable merge if there are diff --git a/binutils/objcopy.c b/binutils/objcopy.c index 36952ecd6f5..ccb5e12798e 100644 --- a/binutils/objcopy.c +++ b/binutils/objcopy.c @@ -2141,7 +2141,7 @@ merge_gnu_build_notes (bfd * abfd, asection * sec, bfd_size_type size, bfd_byte represent a single external reloc. Unfortunately the current BFD API does not handle deleting relocs in such situations very well and so it is unsafe to proceed. */ - if (relcount > sec->reloc_count) + if ((unsigned long) relcount > sec->reloc_count) goto done; /* Eliminate the duplicates. */