]> git.ipfire.org Git - thirdparty/binutils-gdb.git/blobdiff - bfd/elfcore.h
Update year range in copyright notice of binutils files
[thirdparty/binutils-gdb.git] / bfd / elfcore.h
index 51dfec526f4b3b6ba2fb39b8fe61c7c564910fa2..3015e582ef05b24c58aef4479483c762559c7429 100644 (file)
-/* Core file support */
-
-#ifdef HAVE_PROCFS             /* Some core file support requires host /proc files */
-#include <sys/procfs.h>
-#else
-#define bfd_prstatus(abfd, descdata, descsz, filepos) true
-#define bfd_fpregset(abfd, descdata, descsz, filepos) true
-#define bfd_prpsinfo(abfd, descdata, descsz, filepos) true
-#endif
+/* ELF core file support for BFD.
+   Copyright (C) 1995-2021 Free Software Foundation, Inc.
 
-#ifdef HAVE_PROCFS
+   This file is part of BFD, the Binary File Descriptor library.
 
-static boolean
-bfd_prstatus (abfd, descdata, descsz, filepos)
-     bfd *abfd;
-     char *descdata;
-     int descsz;
-     long filepos;
-{
-  asection *newsect;
-  prstatus_t *status = (prstatus_t *) 0;
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General Public License as published by
+   the Free Software Foundation; either version 3 of the License, or
+   (at your option) any later version.
 
-  if (descsz == sizeof (prstatus_t))
-    {
-      newsect = bfd_make_section (abfd, ".reg");
-      if (newsect == NULL)
-       return false;
-      newsect->_raw_size = sizeof (status->pr_reg);
-      newsect->filepos = filepos + (long) &status->pr_reg;
-      newsect->flags = SEC_HAS_CONTENTS;
-      newsect->alignment_power = 2;
-      if ((core_prstatus (abfd) = bfd_alloc (abfd, descsz)) != NULL)
-       {
-         memcpy (core_prstatus (abfd), descdata, descsz);
-       }
-    }
-  return true;
-}
+   This program is distributed in the hope that it will be useful,
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+   GNU General Public License for more details.
 
-/* Stash a copy of the prpsinfo structure away for future use. */
+   You should have received a copy of the GNU General Public License
+   along with this program; if not, write to the Free Software
+   Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston,
+   MA 02110-1301, USA.  */
 
-static boolean
-bfd_prpsinfo (abfd, descdata, descsz, filepos)
-     bfd *abfd;
-     char *descdata;
-     int descsz;
-     long filepos;
+char*
+elf_core_file_failing_command (bfd *abfd)
 {
-  if (descsz == sizeof (prpsinfo_t))
-    {
-      if ((core_prpsinfo (abfd) = bfd_alloc (abfd, descsz)) == NULL)
-       {
-         bfd_set_error (bfd_error_no_memory);
-         return false;
-       }
-      memcpy (core_prpsinfo (abfd), descdata, descsz);
-    }
-  return true;
+  return elf_tdata (abfd)->core->command;
 }
 
-static boolean
-bfd_fpregset (abfd, descdata, descsz, filepos)
-     bfd *abfd;
-     char *descdata;
-     int descsz;
-     long filepos;
+int
+elf_core_file_failing_signal (bfd *abfd)
 {
-  asection *newsect;
-
-  newsect = bfd_make_section (abfd, ".reg2");
-  if (newsect == NULL)
-    return false;
-  newsect->_raw_size = descsz;
-  newsect->filepos = filepos;
-  newsect->flags = SEC_HAS_CONTENTS;
-  newsect->alignment_power = 2;
-  return true;
+  return elf_tdata (abfd)->core->signal;
 }
 
-#endif /* HAVE_PROCFS */
-
-/* Return a pointer to the args (including the command name) that were
-   seen by the program that generated the core dump.  Note that for
-   some reason, a spurious space is tacked onto the end of the args
-   in some (at least one anyway) implementations, so strip it off if
-   it exists. */
-
-char *
-elf_core_file_failing_command (abfd)
-     bfd *abfd;
-{
-#ifdef HAVE_PROCFS
-  if (core_prpsinfo (abfd))
-    {
-      prpsinfo_t *p = core_prpsinfo (abfd);
-      char *scan = p->pr_psargs;
-      while (*scan++)
-       {;
-       }
-      scan -= 2;
-      if ((scan > p->pr_psargs) && (*scan == ' '))
-       {
-         *scan = '\000';
-       }
-      return p->pr_psargs;
-    }
-#endif
-  return NULL;
-}
-
-/* Return the number of the signal that caused the core dump.  Presumably,
-   since we have a core file, we got a signal of some kind, so don't bother
-   checking the other process status fields, just return the signal number.
-   */
-
 int
-elf_core_file_failing_signal (abfd)
-     bfd *abfd;
+elf_core_file_pid (bfd *abfd)
 {
-#ifdef HAVE_PROCFS
-  if (core_prstatus (abfd))
-    {
-      return ((prstatus_t *) (core_prstatus (abfd)))->pr_cursig;
-    }
-#endif
-  return -1;
+  return elf_tdata (abfd)->core->pid;
 }
 
-/* Check to see if the core file could reasonably be expected to have
-   come for the current executable file.  Note that by default we return
-   true unless we find something that indicates that there might be a
-   problem.
-   */
-
-boolean
-elf_core_file_matches_executable_p (core_bfd, exec_bfd)
-     bfd *core_bfd;
-     bfd *exec_bfd;
+bfd_boolean
+elf_core_file_matches_executable_p (bfd *core_bfd, bfd *exec_bfd)
 {
-#ifdef HAVE_PROCFS
-  char *corename;
-  char *execname;
-#endif
+  char* corename;
 
-  /* First, xvecs must match since both are ELF files for the same target. */
+  /* xvecs must match if both are ELF files for the same target.  */
 
   if (core_bfd->xvec != exec_bfd->xvec)
     {
       bfd_set_error (bfd_error_system_call);
-      return false;
-    }
-
-#ifdef HAVE_PROCFS
-
-  /* If no prpsinfo, just return true.  Otherwise, grab the last component
-     of the exec'd pathname from the prpsinfo. */
-
-  if (core_prpsinfo (core_bfd))
-    {
-      corename = (((prpsinfo_t *) core_prpsinfo (core_bfd))->pr_fname);
-    }
-  else
-    {
-      return true;
+      return FALSE;
     }
 
-  /* Find the last component of the executable pathname. */
-
-  if ((execname = strrchr (exec_bfd->filename, '/')) != NULL)
-    {
-      execname++;
-    }
-  else
+  /* If both BFDs have identical build-ids, then they match.  */
+  if (core_bfd->build_id != NULL
+      && exec_bfd->build_id != NULL
+      && core_bfd->build_id->size == exec_bfd->build_id->size
+      && memcmp (core_bfd->build_id->data, exec_bfd->build_id->data,
+                core_bfd->build_id->size) == 0)
+    return TRUE;
+
+  /* See if the name in the corefile matches the executable name.  */
+  corename = elf_tdata (core_bfd)->core->program;
+  if (corename != NULL)
     {
-      execname = (char *) exec_bfd->filename;
-    }
-
-  /* See if they match */
+      const char* execname = strrchr (bfd_get_filename (exec_bfd), '/');
 
-  return strcmp (execname, corename) ? false : true;
+      execname = execname ? execname + 1 : bfd_get_filename (exec_bfd);
 
-#else
-
-  return true;
-
-#endif /* HAVE_PROCFS */
-}
-
-/* ELF core files contain a segment of type PT_NOTE, that holds much of
-   the information that would normally be available from the /proc interface
-   for the process, at the time the process dumped core.  Currently this
-   includes copies of the prstatus, prpsinfo, and fpregset structures.
-
-   Since these structures are potentially machine dependent in size and
-   ordering, bfd provides two levels of support for them.  The first level,
-   available on all machines since it does not require that the host
-   have /proc support or the relevant include files, is to create a bfd
-   section for each of the prstatus, prpsinfo, and fpregset structures,
-   without any interpretation of their contents.  With just this support,
-   the bfd client will have to interpret the structures itself.  Even with
-   /proc support, it might want these full structures for it's own reasons.
-
-   In the second level of support, where HAVE_PROCFS is defined, bfd will
-   pick apart the structures to gather some additional information that
-   clients may want, such as the general register set, the name of the
-   exec'ed file and its arguments, the signal (if any) that caused the
-   core dump, etc.
-
-   */
-
-static boolean
-elf_corefile_note (abfd, hdr)
-     bfd *abfd;
-     Elf_Internal_Phdr *hdr;
-{
-  Elf_External_Note *x_note_p; /* Elf note, external form */
-  Elf_Internal_Note i_note;    /* Elf note, internal form */
-  char *buf = NULL;            /* Entire note segment contents */
-  char *namedata;              /* Name portion of the note */
-  char *descdata;              /* Descriptor portion of the note */
-  char *sectname;              /* Name to use for new section */
-  long filepos;                        /* File offset to descriptor data */
-  asection *newsect;
-
-  if (hdr->p_filesz > 0
-      && (buf = (char *) malloc (hdr->p_filesz)) != NULL
-      && bfd_seek (abfd, hdr->p_offset, SEEK_SET) != -1
-      && bfd_read ((PTR) buf, hdr->p_filesz, 1, abfd) == hdr->p_filesz)
-    {
-      x_note_p = (Elf_External_Note *) buf;
-      while ((char *) x_note_p < (buf + hdr->p_filesz))
-       {
-         i_note.namesz = bfd_h_get_32 (abfd, (bfd_byte *) x_note_p->namesz);
-         i_note.descsz = bfd_h_get_32 (abfd, (bfd_byte *) x_note_p->descsz);
-         i_note.type = bfd_h_get_32 (abfd, (bfd_byte *) x_note_p->type);
-         namedata = x_note_p->name;
-         descdata = namedata + BFD_ALIGN (i_note.namesz, 4);
-         filepos = hdr->p_offset + (descdata - buf);
-         switch (i_note.type)
-           {
-           case NT_PRSTATUS:
-             /* process descdata as prstatus info */
-             if (! bfd_prstatus (abfd, descdata, i_note.descsz, filepos))
-               return false;
-             sectname = ".prstatus";
-             break;
-           case NT_FPREGSET:
-             /* process descdata as fpregset info */
-             if (! bfd_fpregset (abfd, descdata, i_note.descsz, filepos))
-               return false;
-             sectname = ".fpregset";
-             break;
-           case NT_PRPSINFO:
-             /* process descdata as prpsinfo */
-             if (! bfd_prpsinfo (abfd, descdata, i_note.descsz, filepos))
-               return false;
-             sectname = ".prpsinfo";
-             break;
-           default:
-             /* Unknown descriptor, just ignore it. */
-             sectname = NULL;
-             break;
-           }
-         if (sectname != NULL)
-           {
-             newsect = bfd_make_section (abfd, sectname);
-             if (newsect == NULL)
-               return false;
-             newsect->_raw_size = i_note.descsz;
-             newsect->filepos = filepos;
-             newsect->flags = SEC_ALLOC | SEC_HAS_CONTENTS;
-             newsect->alignment_power = 2;
-           }
-         x_note_p = (Elf_External_Note *)
-           (descdata + BFD_ALIGN (i_note.descsz, 4));
-       }
-    }
-  if (buf != NULL)
-    {
-      free (buf);
+      if (strcmp (execname, corename) != 0)
+       return FALSE;
     }
-  else if (hdr->p_filesz > 0)
-    {
-      bfd_set_error (bfd_error_no_memory);
-      return false;
-    }
-  return true;
 
+  return TRUE;
 }
 
 /*  Core files are simply standard ELF formatted files that partition
@@ -297,89 +81,60 @@ elf_corefile_note (abfd, hdr)
     register set) and the floating point register set are stored in a
     segment of type PT_NOTE.  We handcraft a couple of extra bfd sections
     that allow standard bfd access to the general registers (.reg) and the
-    floating point registers (.reg2).
-
- */
+    floating point registers (.reg2).  */
 
-const bfd_target *
-elf_core_file_p (abfd)
-     bfd *abfd;
+bfd_cleanup
+elf_core_file_p (bfd *abfd)
 {
-  Elf_External_Ehdr x_ehdr;    /* Elf file header, external form */
-  Elf_Internal_Ehdr *i_ehdrp;  /* Elf file header, internal form */
-  Elf_External_Phdr x_phdr;    /* Program header table entry, external form */
-  Elf_Internal_Phdr *i_phdrp;  /* Program header table, internal form */
+  Elf_External_Ehdr x_ehdr;    /* Elf file header, external form.  */
+  Elf_Internal_Ehdr *i_ehdrp;  /* Elf file header, internal form.  */
+  Elf_Internal_Phdr *i_phdrp;  /* Elf program header, internal form.  */
   unsigned int phindex;
-  struct elf_backend_data *ebd;
+  const struct elf_backend_data *ebd;
+  bfd_size_type amt;
 
   /* Read in the ELF header in external format.  */
-
-  if (bfd_read ((PTR) & x_ehdr, sizeof (x_ehdr), 1, abfd) != sizeof (x_ehdr))
+  if (bfd_bread (&x_ehdr, sizeof (x_ehdr), abfd) != sizeof (x_ehdr))
     {
       if (bfd_get_error () != bfd_error_system_call)
-       bfd_set_error (bfd_error_wrong_format);
-      return NULL;
-    }
-
-  /* Now check to see if we have a valid ELF file, and one that BFD can
-     make use of.  The magic number must match, the address size ('class')
-     and byte-swapping must match our XVEC entry, and it must have a
-     program header table (FIXME: See comments re segments at top of this
-     file). */
-
-  if (elf_file_p (&x_ehdr) == false)
-    {
-    wrong:
-      bfd_set_error (bfd_error_wrong_format);
-      return NULL;
+       goto wrong;
+      else
+       goto fail;
     }
 
-  /* FIXME, Check EI_VERSION here !  */
+  /* Check the magic number.  */
+  if (! elf_file_p (&x_ehdr))
+    goto wrong;
 
-  {
-#if ARCH_SIZE == 32
-    int desired_address_size = ELFCLASS32;
-#endif
-#if ARCH_SIZE == 64
-    int desired_address_size = ELFCLASS64;
-#endif
+  /* FIXME: Check EI_VERSION here !  */
 
-    if (x_ehdr.e_ident[EI_CLASS] != desired_address_size)
-      goto wrong;
-  }
+  /* Check the address size ("class").  */
+  if (x_ehdr.e_ident[EI_CLASS] != ELFCLASS)
+    goto wrong;
 
-  /* Switch xvec to match the specified byte order.  */
+  /* Check the byteorder.  */
   switch (x_ehdr.e_ident[EI_DATA])
     {
-    case ELFDATA2MSB:          /* Big-endian */
-      if (abfd->xvec->byteorder_big_p == false)
+    case ELFDATA2MSB:          /* Big-endian */
+      if (! bfd_big_endian (abfd))
        goto wrong;
       break;
-    case ELFDATA2LSB:          /* Little-endian */
-      if (abfd->xvec->byteorder_big_p == true)
+    case ELFDATA2LSB:          /* Little-endian */
+      if (! bfd_little_endian (abfd))
        goto wrong;
       break;
-    case ELFDATANONE:          /* No data encoding specified */
-    default:                   /* Unknown data encoding specified */
+    default:
       goto wrong;
     }
 
-  /* Allocate an instance of the elf_obj_tdata structure and hook it up to
-     the tdata pointer in the bfd. */
-
-  elf_tdata (abfd) =
-    (struct elf_obj_tdata *) bfd_zalloc (abfd, sizeof (struct elf_obj_tdata));
-  if (elf_tdata (abfd) == NULL)
-    {
-      bfd_set_error (bfd_error_no_memory);
-      return NULL;
-    }
-
-  /* FIXME, `wrong' returns from this point onward, leak memory.  */
+  /* Give abfd an elf_obj_tdata.  */
+  if (! (*abfd->xvec->_bfd_set_format[bfd_core]) (abfd))
+    goto fail;
 
-  /* Now that we know the byte order, swap in the rest of the header */
+  /* Swap in the rest of the header, now that we have the byte order.  */
   i_ehdrp = elf_elfheader (abfd);
   elf_swap_ehdr_in (abfd, &x_ehdr, i_ehdrp);
+
 #if DEBUG & 1
   elf_debug_file (i_ehdrp);
 #endif
@@ -388,9 +143,12 @@ elf_core_file_p (abfd)
 
   /* Check that the ELF e_machine field matches what this particular
      BFD format expects.  */
+
   if (ebd->elf_machine_code != i_ehdrp->e_machine
-      && (ebd->elf_machine_alt1 == 0 || i_ehdrp->e_machine != ebd->elf_machine_alt1)
-      && (ebd->elf_machine_alt2 == 0 || i_ehdrp->e_machine != ebd->elf_machine_alt2))
+      && (ebd->elf_machine_alt1 == 0
+         || i_ehdrp->e_machine != ebd->elf_machine_alt1)
+      && (ebd->elf_machine_alt2 == 0
+         || i_ehdrp->e_machine != ebd->elf_machine_alt2))
     {
       const bfd_target * const *target_ptr;
 
@@ -399,14 +157,21 @@ elf_core_file_p (abfd)
 
       /* This is the generic ELF target.  Let it match any ELF target
         for which we do not have a specific backend.  */
+
       for (target_ptr = bfd_target_vector; *target_ptr != NULL; target_ptr++)
        {
-         struct elf_backend_data *back;
+         const struct elf_backend_data *back;
 
          if ((*target_ptr)->flavour != bfd_target_elf_flavour)
            continue;
-         back = (struct elf_backend_data *) (*target_ptr)->backend_data;
-         if (back->elf_machine_code == i_ehdrp->e_machine)
+         back = xvec_get_elf_backend_data (*target_ptr);
+         if (back->s->arch_size != ARCH_SIZE)
+           continue;
+         if (back->elf_machine_code == i_ehdrp->e_machine
+             || (back->elf_machine_alt1 != 0
+                 && i_ehdrp->e_machine == back->elf_machine_alt1)
+             || (back->elf_machine_alt2 != 0
+                 && i_ehdrp->e_machine == back->elf_machine_alt2))
            {
              /* target_ptr is an ELF backend which matches this
                 object file, so reject the generic ELF target.  */
@@ -416,51 +181,246 @@ elf_core_file_p (abfd)
     }
 
   /* If there is no program header, or the type is not a core file, then
-     we are hosed. */
+     we are hosed.  */
   if (i_ehdrp->e_phoff == 0 || i_ehdrp->e_type != ET_CORE)
     goto wrong;
 
-  /* Allocate space for a copy of the program header table in
-     internal form, seek to the program header table in the file,
-     read it in, and convert it to internal form.  As a simple sanity
-     check, verify that the what BFD thinks is the size of each program
-     header table entry actually matches the size recorded in the file. */
+  /* Does BFD's idea of the phdr size match the size
+     recorded in the file? */
+  if (i_ehdrp->e_phentsize != sizeof (Elf_External_Phdr))
+    goto wrong;
 
-  if (i_ehdrp->e_phentsize != sizeof (x_phdr))
+  /* If the program header count is PN_XNUM(0xffff), the actual
+     count is in the first section header.  */
+  if (i_ehdrp->e_shoff != 0 && i_ehdrp->e_phnum == PN_XNUM)
+    {
+      Elf_External_Shdr x_shdr;
+      Elf_Internal_Shdr i_shdr;
+      file_ptr where = (file_ptr) i_ehdrp->e_shoff;
+
+      /* Seek to the section header table in the file.  */
+      if (bfd_seek (abfd, where, SEEK_SET) != 0)
+       goto fail;
+
+      /* Read the first section header at index 0, and convert to internal
+        form.  */
+      if (bfd_bread (&x_shdr, sizeof (x_shdr), abfd) != sizeof (x_shdr))
+       goto fail;
+      elf_swap_shdr_in (abfd, &x_shdr, &i_shdr);
+
+      if (i_shdr.sh_info != 0)
+       {
+         i_ehdrp->e_phnum = i_shdr.sh_info;
+         if (i_ehdrp->e_phnum != i_shdr.sh_info)
+           goto wrong;
+       }
+    }
+
+  /* Sanity check that we can read all of the program headers.
+     It ought to be good enough to just read the last one.  */
+  if (i_ehdrp->e_phnum > 1)
+    {
+      Elf_External_Phdr x_phdr;
+      Elf_Internal_Phdr i_phdr;
+      file_ptr where;
+
+      /* Check that we don't have a totally silly number of
+        program headers.  */
+      if (i_ehdrp->e_phnum > (unsigned int) -1 / sizeof (x_phdr)
+         || i_ehdrp->e_phnum > (unsigned int) -1 / sizeof (i_phdr))
+       goto wrong;
+
+      where = (file_ptr)(i_ehdrp->e_phoff + (i_ehdrp->e_phnum - 1) * sizeof (x_phdr));
+      if ((bfd_size_type) where <= i_ehdrp->e_phoff)
+       goto wrong;
+
+      if (bfd_seek (abfd, where, SEEK_SET) != 0)
+       goto fail;
+      if (bfd_bread (&x_phdr, sizeof (x_phdr), abfd) != sizeof (x_phdr))
+       goto fail;
+    }
+
+  /* Move to the start of the program headers.  */
+  if (bfd_seek (abfd, (file_ptr) i_ehdrp->e_phoff, SEEK_SET) != 0)
     goto wrong;
-  i_phdrp = (Elf_Internal_Phdr *)
-    bfd_alloc (abfd, sizeof (*i_phdrp) * i_ehdrp->e_phnum);
+
+  /* Allocate space for the program headers.  */
+  amt = sizeof (*i_phdrp) * i_ehdrp->e_phnum;
+  i_phdrp = (Elf_Internal_Phdr *) bfd_alloc (abfd, amt);
   if (!i_phdrp)
+    goto fail;
+
+  elf_tdata (abfd)->phdr = i_phdrp;
+
+  /* Read and convert to internal form.  */
+  for (phindex = 0; phindex < i_ehdrp->e_phnum; ++phindex)
     {
-      bfd_set_error (bfd_error_no_memory);
-      return NULL;
+      Elf_External_Phdr x_phdr;
+
+      if (bfd_bread (&x_phdr, sizeof (x_phdr), abfd) != sizeof (x_phdr))
+       goto fail;
+
+      elf_swap_phdr_in (abfd, &x_phdr, i_phdrp + phindex);
     }
-  if (bfd_seek (abfd, i_ehdrp->e_phoff, SEEK_SET) == -1)
-    return NULL;
-  for (phindex = 0; phindex < i_ehdrp->e_phnum; phindex++)
+
+  /* Set the machine architecture.  Do this before processing the
+     program headers since we need to know the architecture type
+     when processing the notes of some systems' core files.  */
+  if (! bfd_default_set_arch_mach (abfd, ebd->arch, 0)
+      /* It's OK if this fails for the generic target.  */
+      && ebd->elf_machine_code != EM_NONE)
+    goto fail;
+
+  /* Let the backend double check the format and override global
+     information.  We do this before processing the program headers
+     to allow the correct machine (as opposed to just the default
+     machine) to be set, making it possible for grok_prstatus and
+     grok_psinfo to rely on the mach setting.  */
+  if (ebd->elf_backend_object_p != NULL
+      && ! ebd->elf_backend_object_p (abfd))
+    goto wrong;
+
+  /* Process each program header.  */
+  for (phindex = 0; phindex < i_ehdrp->e_phnum; ++phindex)
+    if (! bfd_section_from_phdr (abfd, i_phdrp + phindex, (int) phindex))
+      goto fail;
+
+  /* Check for core truncation.  */
+  {
+    bfd_size_type high = 0;
+    struct stat statbuf;
+    for (phindex = 0; phindex < i_ehdrp->e_phnum; ++phindex)
+      {
+       Elf_Internal_Phdr *p = i_phdrp + phindex;
+       if (p->p_filesz)
+         {
+           bfd_size_type current = p->p_offset + p->p_filesz;
+           if (high < current)
+             high = current;
+         }
+      }
+    if (bfd_stat (abfd, &statbuf) == 0)
+      {
+       if ((bfd_size_type) statbuf.st_size < high)
+         {
+           _bfd_error_handler
+             /* xgettext:c-format */
+             (_("warning: %pB is truncated: expected core file "
+                "size >= %" PRIu64 ", found: %" PRIu64),
+              abfd, (uint64_t) high, (uint64_t) statbuf.st_size);
+         }
+      }
+  }
+
+  /* Save the entry point from the ELF header.  */
+  abfd->start_address = i_ehdrp->e_entry;
+  return _bfd_no_cleanup;
+
+ wrong:
+  bfd_set_error (bfd_error_wrong_format);
+ fail:
+  return NULL;
+}
+
+/* Attempt to find a build-id in a core file from the core file BFD.
+   OFFSET is the file offset to a PT_LOAD segment that may contain
+   the build-id note.  Returns TRUE upon success, FALSE otherwise.  */
+
+bfd_boolean
+NAME(_bfd_elf, core_find_build_id)
+  (bfd *abfd,
+   bfd_vma offset)
+{
+  Elf_External_Ehdr x_ehdr;    /* Elf file header, external form.   */
+  Elf_Internal_Ehdr i_ehdr;    /* Elf file header, internal form.   */
+  Elf_Internal_Phdr *i_phdr;
+  unsigned int i;
+  size_t amt;
+
+  /* Seek to the position of the segment at OFFSET.  */
+  if (bfd_seek (abfd, offset, SEEK_SET) != 0)
+    goto fail;
+
+  /* Read in the ELF header in external format.  */
+  if (bfd_bread (&x_ehdr, sizeof (x_ehdr), abfd) != sizeof (x_ehdr))
     {
-      if (bfd_read ((PTR) & x_phdr, sizeof (x_phdr), 1, abfd)
-         != sizeof (x_phdr))
-       return NULL;
-      elf_swap_phdr_in (abfd, &x_phdr, i_phdrp + phindex);
+      if (bfd_get_error () != bfd_error_system_call)
+       goto wrong;
+      else
+       goto fail;
+    }
+
+  /* Now check to see if we have a valid ELF file, and one that BFD can
+     make use of.  The magic number must match, the address size ('class')
+     and byte-swapping must match our XVEC entry, and it must have a
+     section header table (FIXME: See comments re sections at top of this
+     file).  */
+  if (! elf_file_p (&x_ehdr)
+      || x_ehdr.e_ident[EI_VERSION] != EV_CURRENT
+      || x_ehdr.e_ident[EI_CLASS] != ELFCLASS)
+    goto wrong;
+
+  /* Check that file's byte order matches xvec's.  */
+  switch (x_ehdr.e_ident[EI_DATA])
+    {
+    case ELFDATA2MSB:          /* Big-endian.  */
+      if (! bfd_header_big_endian (abfd))
+       goto wrong;
+      break;
+    case ELFDATA2LSB:          /* Little-endian.  */
+      if (! bfd_header_little_endian (abfd))
+       goto wrong;
+      break;
+    case ELFDATANONE:          /* No data encoding specified.  */
+    default:                   /* Unknown data encoding specified . */
+      goto wrong;
     }
 
-  /* Once all of the program headers have been read and converted, we
-     can start processing them. */
+  elf_swap_ehdr_in (abfd, &x_ehdr, &i_ehdr);
+#if DEBUG
+  elf_debug_file (&i_ehdr);
+#endif
+
+  if (i_ehdr.e_phentsize != sizeof (Elf_External_Phdr) || i_ehdr.e_phnum == 0)
+    goto fail;
 
-  for (phindex = 0; phindex < i_ehdrp->e_phnum; phindex++)
+  /* Read in program headers.  */
+  if (_bfd_mul_overflow (i_ehdr.e_phnum, sizeof (*i_phdr), &amt))
     {
-      bfd_section_from_phdr (abfd, i_phdrp + phindex, phindex);
-      if ((i_phdrp + phindex)->p_type == PT_NOTE)
+      bfd_set_error (bfd_error_file_too_big);
+      goto fail;
+    }
+  i_phdr = (Elf_Internal_Phdr *) bfd_alloc (abfd, amt);
+  if (i_phdr == NULL)
+    goto fail;
+
+  if (bfd_seek (abfd, (file_ptr) (offset + i_ehdr.e_phoff), SEEK_SET) != 0)
+    goto fail;
+
+  /* Read in program headers and parse notes.  */
+  for (i = 0; i < i_ehdr.e_phnum; ++i, ++i_phdr)
+    {
+      Elf_External_Phdr x_phdr;
+
+      if (bfd_bread (&x_phdr, sizeof (x_phdr), abfd) != sizeof (x_phdr))
+       goto fail;
+      elf_swap_phdr_in (abfd, &x_phdr, i_phdr);
+
+      if (i_phdr->p_type == PT_NOTE && i_phdr->p_filesz > 0)
        {
-         if (! elf_corefile_note (abfd, i_phdrp + phindex))
-           return NULL;
+         elf_read_notes (abfd, offset + i_phdr->p_offset,
+                         i_phdr->p_filesz, i_phdr->p_align);
+         if (abfd->build_id != NULL)
+           return TRUE;
        }
     }
 
-  /* Remember the entry point specified in the ELF file header. */
-
-  bfd_get_start_address (abfd) = i_ehdrp->e_entry;
+  /* Having gotten this far, we have a valid ELF section, but no
+     build-id was found.  */
+  goto fail;
 
-  return abfd->xvec;
+ wrong:
+  bfd_set_error (bfd_error_wrong_format);
+ fail:
+  return FALSE;
 }