From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 78883 invoked by alias); 27 Feb 2020 17:40:31 -0000 Mailing-List: contact cygwin-cvs-help@cygwin.com; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: cygwin-cvs-owner@cygwin.com Received: (qmail 78842 invoked by uid 9795); 27 Feb 2020 17:40:31 -0000 Date: Thu, 27 Feb 2020 17:40:00 -0000 Message-ID: <20200227174031.78840.qmail@sourceware.org> Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Jon TURNEY To: cygwin-cvs@sourceware.org Subject: [newlib-cygwin] Cygwin: Update dumper for bfd API changes X-Act-Checkin: newlib-cygwin X-Git-Author: Jon Turney X-Git-Refname: refs/heads/master X-Git-Oldrev: c13cdfd00ed09d23e5cf1864033a1bfd54bc3aab X-Git-Newrev: ba2f251d439294c7087f3a38a8d407c95cdc5c1e X-SW-Source: 2020-q1/txt/msg00090.txt https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=ba2f251d439294c7087f3a38a8d407c95cdc5c1e commit ba2f251d439294c7087f3a38a8d407c95cdc5c1e Author: Jon Turney Date: Wed Feb 26 18:48:51 2020 +0000 Cygwin: Update dumper for bfd API changes Update dumper for bfd API changes in binutils 2.34 libbfd doesn't guarantee API stability, so we've just been lucky this hasn't broken more often. See binutils commit fd361982. Diff: --- winsup/utils/dumper.cc | 30 ++++++++++++++++++++++-------- winsup/utils/parse_pe.cc | 4 ++++ 2 files changed, 26 insertions(+), 8 deletions(-) diff --git a/winsup/utils/dumper.cc b/winsup/utils/dumper.cc index f71bdda..226c228 100644 --- a/winsup/utils/dumper.cc +++ b/winsup/utils/dumper.cc @@ -39,6 +39,20 @@ #define NOTE_NAME_SIZE 16 +#ifdef bfd_get_section_size +/* for bfd < 2.34 */ +#define get_section_name(abfd, sect) bfd_get_section_name (abfd, sect) +#define get_section_size(sect) bfd_get_section_size(sect) +#define set_section_size(abfd, sect, size) bfd_set_section_size(abfd, sect, size) +#define set_section_flags(abfd, sect, flags) bfd_set_section_flags(abfd, sect, flags) +#else +/* otherwise bfd >= 2.34 */ +#define get_section_name(afbd, sect) bfd_section_name (sect) +#define get_section_size(sect) bfd_section_size(sect) +#define set_section_size(abfd, sect, size) bfd_set_section_size(sect, size) +#define set_section_flags(abfd, sect, flags) bfd_set_section_flags(sect, flags) +#endif + typedef struct _note_header { Elf_External_Note elf_note_header; @@ -131,7 +145,7 @@ dumper::sane () void print_section_name (bfd* abfd, asection* sect, PTR obj) { - deb_printf (" %s", bfd_get_section_name (abfd, sect)); + deb_printf (" %s", get_section_name (abfd, sect)); } void @@ -712,10 +726,10 @@ dumper::prepare_core_dump () if (p->type == pr_ent_module && status_section != NULL) { - if (!bfd_set_section_size (core_bfd, - status_section, - (bfd_get_section_size (status_section) - + sect_size))) + if (!set_section_size (core_bfd, + status_section, + (get_section_size (status_section) + + sect_size))) { bfd_perror ("resizing status section"); goto failed; @@ -738,8 +752,8 @@ dumper::prepare_core_dump () goto failed; } - if (!bfd_set_section_flags (core_bfd, new_section, sect_flags) || - !bfd_set_section_size (core_bfd, new_section, sect_size)) + if (!set_section_flags (core_bfd, new_section, sect_flags) || + !set_section_size (core_bfd, new_section, sect_size)) { bfd_perror ("setting section attributes"); goto failed; @@ -823,7 +837,7 @@ dumper::write_core_dump () deb_printf ("writing section type=%u base=%p size=%p flags=%08x\n", p->type, p->section->vma, - bfd_get_section_size (p->section), + get_section_size (p->section), p->section->flags); switch (p->type) diff --git a/winsup/utils/parse_pe.cc b/winsup/utils/parse_pe.cc index 2a38863..90b5c0b 100644 --- a/winsup/utils/parse_pe.cc +++ b/winsup/utils/parse_pe.cc @@ -25,6 +25,10 @@ #include "dumper.h" +#ifndef bfd_get_section_size +#define bfd_get_section_size(sect) bfd_section_size(sect) +#endif + int exclusion::add (LPBYTE mem_base, SIZE_T mem_size) {