[Bug binutils/22485] New: Bad initialization for Filedata struct in readelf.c

2017-11-23 Thread kryukov at frtk dot ru
https://sourceware.org/bugzilla/show_bug.cgi?id=22485

Bug ID: 22485
   Summary: Bad initialization for Filedata struct in readelf.c
   Product: binutils
   Version: 2.30 (HEAD)
Status: UNCONFIRMED
  Severity: normal
  Priority: P2
 Component: binutils
  Assignee: unassigned at sourceware dot org
  Reporter: kryukov at frtk dot ru
  Target Milestone: ---

Hi

My project builds GNU Binutils with Clang as a part of CI. Clang warns that
fields of Filedata structure would not be initialized here:

readelf.c:18848:40: error: missing field 'handle' initializer
[-Werror,-Wmissing-field-initializers]
  Filedata thin_filedata = { 0 };

Actually it is a known Clang bug, but they are not fixing it for 5 years
(https://bugs.llvm.org/show_bug.cgi?id=21689), so I prepared a patch where
initialization with { 0 } is replaced with memset.

Please pull it from master branch here:
https://github.com/pavelkryukov/binutils-gdb.git

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/22471] libraries using version scripts can cause undefined reference to linker-defined symbols

2017-11-23 Thread hjl.tools at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=22471

H.J. Lu  changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2017-11-23
 CC||hjl.tools at gmail dot com
Summary|libraries using version |libraries using version
   |scripts can cause undefined |scripts can cause undefined
   |reference to symbol |reference to linker-defined
   |'__bss_start'   |symbols
 Ever confirmed|0   |1

--- Comment #1 from H.J. Lu  ---
This can happen to any linker defined symbols.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/22471] libraries using version scripts can cause undefined reference to linker-defined symbols

2017-11-23 Thread hjl.tools at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=22471

--- Comment #2 from H.J. Lu  ---
Builtin linker scripts for each target define different sets
of symbols.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/22485] Bad initialization for Filedata struct in readelf.c

2017-11-23 Thread kryukov at frtk dot ru
https://sourceware.org/bugzilla/show_bug.cgi?id=22485

Pavel I. Kryukov  changed:

   What|Removed |Added

 CC||nickc at redhat dot com

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/22485] Bad initialization for Filedata struct in readelf.c

2017-11-23 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=22485

--- Comment #1 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Nick Clifton :

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=eb02c04dc3600e18bb06dc1ef8fc3cf9a967d550

commit eb02c04dc3600e18bb06dc1ef8fc3cf9a967d550
Author: Pavel I. Kryukov 
Date:   Thu Nov 23 15:53:04 2017 +

Avoid bogus compile time warning from clang by initialising local data
structure using memset.

PR 22485
* readelf.c (process_archive): Use memset to initiaise
thin_filedata structure.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/22485] Bad initialization for Filedata struct in readelf.c

2017-11-23 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=22485

Nick Clifton  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #2 from Nick Clifton  ---
Patch committed.  Thanks.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/22471] libraries using version scripts can cause undefined reference to linker-defined symbols

2017-11-23 Thread hjl.tools at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=22471

--- Comment #3 from H.J. Lu  ---
Created attachment 10631
  --> https://sourceware.org/bugzilla/attachment.cgi?id=10631&action=edit
I am testing this patch.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/22443] Global buffer overflow in _bfd_elf_get_symbol_version_string

2017-11-23 Thread amodra at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=22443

Alan Modra  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #11 from Alan Modra  ---
Fixed

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/22431] powerpc64 ld segfault when .plt discarded

2017-11-23 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=22431

--- Comment #4 from cvs-commit at gcc dot gnu.org  ---
The binutils-2_29-branch branch has been updated by Alan Modra
:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=e4495167d97f38d4e1f78b518e59e00a5533d768

commit e4495167d97f38d4e1f78b518e59e00a5533d768
Author: Alan Modra 
Date:   Tue Nov 14 18:47:34 2017 +1030

PR22431, powerpc64 ld segfault when .plt discarded

The fix for the PR is to not use input_section->output_section->owner
to get to the output bfd, but use the output bfd directly since it is
available nowadays in struct bfd_link_info.

I thought it worth warning when non-empty dynamic sections are
discarded too, which meant a tweak to one of the ld tests to avoid the
warning.

bfd/
PR 22431
* elf64-ppc.c (ppc64_elf_size_dynamic_sections): Warn on discarding
non-empty dynamic section.
(ppc_build_one_stub): Take elf_gp from output bfd, not output
section owner.
(ppc_size_one_stub, ppc64_elf_next_toc_section): Likewise.
ld/
* testsuite/ld-elf/note-3.t: Don't discard .got.

(cherry picked from commit 06bcf5416feae129e239476ab1408e038bba171d)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/22431] powerpc64 ld segfault when .plt discarded

2017-11-23 Thread amodra at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=22431

Alan Modra  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Alan Modra  ---
Fixed

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils