Update year range in copyright notice of binutils files
[deliverable/binutils-gdb.git] / bfd / versados.c
index f3de87188bb82d46bec74270f674216a7209aee6..369ec2fe3f05cfe48361b50f465b24a23cc988b8 100644 (file)
@@ -1,6 +1,5 @@
 /* BFD back-end for VERSAdos-E objects.
-   Copyright 1995, 1996, 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005,
-   2006, 2007, 2009 Free Software Foundation, Inc.
+   Copyright (C) 1995-2018 Free Software Foundation, Inc.
    Written by Steve Chamberlain of Cygnus Support <sac@cygnus.com>.
 
    Versados is a Motorola trademark.
@@ -58,6 +57,7 @@ struct esdid
 {
   asection *section;           /* Ptr to bfd version.  */
   unsigned char *contents;     /* Used to build image.  */
+  bfd_size_type content_size;  /* The size of the contents buffer.  */
   int pc;
   int relocs;                  /* Reloc count, valid end of pass 1.  */
   int donerel;                 /* Have relocs been translated.  */
@@ -86,8 +86,8 @@ typedef struct versados_data_struct
 tdata_type;
 
 #define VDATA(abfd)       (abfd->tdata.versados_data)
-#define EDATA(abfd, n)    (abfd->tdata.versados_data->e[n])
-#define RDATA(abfd, n)    (abfd->tdata.versados_data->rest[n])
+#define EDATA(abfd, n)    (abfd->tdata.versados_data->e[(n) < 16 ? (n) : 0])
+#define RDATA(abfd, n)    (abfd->tdata.versados_data->rest[(n) < 240 ? (n) : 0])
 
 struct ext_otr
 {
@@ -122,8 +122,8 @@ struct ext_esd
   unsigned char esd_entries[1];
 };
 
-#define ESD_ABS          0
-#define ESD_COMMON       1
+#define ESD_ABS                  0
+#define ESD_COMMON       1
 #define ESD_STD_REL_SEC   2
 #define ESD_SHRT_REL_SEC  3
 #define ESD_XDEF_IN_SEC   4
@@ -149,7 +149,7 @@ versados_mkobject (bfd *abfd)
   if (abfd->tdata.versados_data == NULL)
     {
       bfd_size_type amt = sizeof (tdata_type);
-      tdata_type *tdata = bfd_alloc (abfd, amt);
+      tdata_type *tdata = bfd_zalloc (abfd, amt);
 
       if (tdata == NULL)
        return FALSE;
@@ -182,14 +182,22 @@ versados_new_symbol (bfd *abfd,
   return n;
 }
 
-static int
+static bfd_boolean
 get_record (bfd *abfd, union ext_any *ptr)
 {
   if (bfd_bread (&ptr->size, (bfd_size_type) 1, abfd) != 1
       || (bfd_bread ((char *) ptr + 1, (bfd_size_type) ptr->size, abfd)
          != ptr->size))
-    return 0;
-  return 1;
+    return FALSE;
+
+  {
+    bfd_size_type amt = ptr->size + 1;
+
+    if (amt < sizeof (* ptr))
+      memset ((char *) ptr + amt, 0, sizeof (* ptr) - amt);
+  }
+
+  return TRUE;
 }
 
 static int
@@ -277,7 +285,9 @@ process_esd (bfd *abfd, struct ext_esd *esd, int pass)
 
        case ESD_ABS:
          size = get_4 (&ptr);
+         (void) size;
          start = get_4 (&ptr);
+         (void) start;
          break;
        case ESD_STD_REL_SEC:
        case ESD_SHRT_REL_SEC:
@@ -285,7 +295,8 @@ process_esd (bfd *abfd, struct ext_esd *esd, int pass)
          sec->flags |= SEC_ALLOC;
          break;
        case ESD_XDEF_IN_ABS:
-         sec = (asection *) & bfd_abs_section;
+         sec = bfd_abs_section_ptr;
+         /* Fall through.  */
        case ESD_XDEF_IN_SEC:
          {
            int snum = VDATA (abfd)->def_idx++;
@@ -334,13 +345,13 @@ reloc_howto_type versados_howto_table[] =
 };
 
 static int
-get_offset (int len, unsigned char *ptr)
+get_offset (unsigned int len, unsigned char *ptr)
 {
   int val = 0;
 
   if (len)
     {
-      int i;
+      unsigned int i;
 
       val = *ptr++;
       if (val & 0x80)
@@ -363,10 +374,18 @@ process_otr (bfd *abfd, struct ext_otr *otr, int pass)
   | (otr->map[2] << 8)
   | (otr->map[3] << 0);
 
-  struct esdid *esdid = &EDATA (abfd, otr->esdid - 1);
-  unsigned char *contents = esdid->contents;
-  int need_contents = 0;
-  unsigned int dst_idx = esdid->pc;
+  struct esdid *esdid;
+  unsigned char *contents;
+  bfd_boolean need_contents = FALSE;
+  unsigned int dst_idx;
+
+  /* PR 17512: file: ac7da425.  */
+  if (otr->esdid == 0)
+    return;
+
+  esdid = &EDATA (abfd, otr->esdid - 1);
+  contents = esdid->contents;
+  dst_idx = esdid->pc;
 
   for (shift = ((unsigned long) 1 << 31); shift && srcp < endp; shift >>= 1)
     {
@@ -375,9 +394,13 @@ process_otr (bfd *abfd, struct ext_otr *otr, int pass)
          int flag = *srcp++;
          int esdids = (flag >> 5) & 0x7;
          int sizeinwords = ((flag >> 3) & 1) ? 2 : 1;
-         int offsetlen = flag & 0x7;
+         unsigned int offsetlen = flag & 0x7;
          int j;
 
+         /* PR 21591: Check for invalid lengths.  */
+         if (srcp + esdids + offsetlen >= endp)
+           return;
+
          if (esdids == 0)
            {
              /* A zero esdid means the new pc is the offset given.  */
@@ -389,8 +412,8 @@ process_otr (bfd *abfd, struct ext_otr *otr, int pass)
              int val = get_offset (offsetlen, srcp + esdids);
 
              if (pass == 1)
-               need_contents = 1;
-             else
+               need_contents = TRUE;
+             else if (contents && dst_idx < esdid->content_size - sizeinwords * 2)
                for (j = 0; j < sizeinwords * 2; j++)
                  {
                    contents[dst_idx + (sizeinwords * 2) - j - 1] = val;
@@ -399,9 +422,9 @@ process_otr (bfd *abfd, struct ext_otr *otr, int pass)
 
              for (j = 0; j < esdids; j++)
                {
-                 int esdid = *srcp++;
+                 int id = *srcp++;
 
-                 if (esdid)
+                 if (id)
                    {
                      int rn = EDATA (abfd, otr->esdid - 1).relocs++;
 
@@ -412,11 +435,14 @@ process_otr (bfd *abfd, struct ext_otr *otr, int pass)
                        }
                      else
                        {
-                         arelent *n =
-                         EDATA (abfd, otr->esdid - 1).section->relocation + rn;
-                         n->address = dst_idx;
+                         arelent *n;
 
-                         n->sym_ptr_ptr = (asymbol **) (size_t) esdid;
+                         /* PR 17512: file: 54f733e0.  */
+                         if (EDATA (abfd, otr->esdid - 1).section == NULL)
+                           continue;
+                         n = EDATA (abfd, otr->esdid - 1).section->relocation + rn;
+                         n->address = dst_idx;
+                         n->sym_ptr_ptr = (asymbol **) (size_t) id;
                          n->addend = 0;
                          n->howto = versados_howto_table + ((j & 1) * 2) + (sizeinwords - 1);
                        }
@@ -428,31 +454,42 @@ process_otr (bfd *abfd, struct ext_otr *otr, int pass)
        }
       else
        {
-         need_contents = 1;
-         if (dst_idx < esdid->section->size)
+         need_contents = TRUE;
+
+         if (esdid->section && contents && dst_idx < esdid->content_size - 1)
            if (pass == 2)
              {
                /* Absolute code, comes in 16 bit lumps.  */
                contents[dst_idx] = srcp[0];
                contents[dst_idx + 1] = srcp[1];
              }
+
          dst_idx += 2;
          srcp += 2;
        }
     }
+
   EDATA (abfd, otr->esdid - 1).pc = dst_idx;
 
   if (!contents && need_contents)
     {
-      bfd_size_type size = esdid->section->size;
-      esdid->contents = bfd_alloc (abfd, size);
+      if (esdid->section)
+       {
+         bfd_size_type size;
+
+         size = esdid->section->size;
+         esdid->contents = bfd_alloc (abfd, size);
+         esdid->content_size = size;
+       }
+      else
+       esdid->contents = NULL;
     }
 }
 
 static bfd_boolean
 versados_scan (bfd *abfd)
 {
-  int loop = 1;
+  bfd_boolean loop = TRUE;
   int i;
   int j;
   int nsecs = 0;
@@ -470,13 +507,13 @@ versados_scan (bfd *abfd)
       union ext_any any;
 
       if (!get_record (abfd, &any))
-       return TRUE;
+       return FALSE;
       switch (any.header.type)
        {
        case VHEADER:
          break;
        case VEND:
-         loop = 0;
+         loop = FALSE;
          break;
        case VESTDEF:
          process_esd (abfd, &any.esd, 1);
@@ -503,7 +540,6 @@ versados_scan (bfd *abfd)
        {
          amt = (bfd_size_type) esdid->relocs * sizeof (arelent);
          esdid->section->relocation = bfd_alloc (abfd, amt);
-
          esdid->pc = 0;
 
          if (esdid->contents)
@@ -562,7 +598,7 @@ versados_scan (bfd *abfd)
 
   VDATA (abfd)->ref_idx = 0;
 
-  return 1;
+  return TRUE;
 }
 
 /* Check whether an existing file is a versados  file.  */
@@ -584,6 +620,13 @@ versados_object_p (bfd *abfd)
       return NULL;
     }
 
+  /* PR 17512: file: 726-2128-0.004.  */
+  if (len < 13)
+    {
+      bfd_set_error (bfd_error_wrong_format);
+      return NULL;
+    }
+
   if (bfd_bread (&ext.type, (bfd_size_type) len, abfd) != len)
     {
       if (bfd_get_error () != bfd_error_system_call)
@@ -651,12 +694,20 @@ versados_get_section_contents (bfd *abfd,
                               file_ptr offset,
                               bfd_size_type count)
 {
+  struct esdid *esdid;
+
   if (!versados_pass_2 (abfd))
     return FALSE;
 
-  memcpy (location,
-         EDATA (abfd, section->target_index).contents + offset,
-         (size_t) count);
+  esdid = &EDATA (abfd, section->target_index);
+
+  if (esdid->contents == NULL
+      || offset < 0
+      || (bfd_size_type) offset > esdid->content_size
+      || offset + count > esdid->content_size)
+    return FALSE;
+
+  memcpy (location, esdid->contents + offset, (size_t) count);
 
   return TRUE;
 }
@@ -757,6 +808,7 @@ versados_canonicalize_reloc (bfd *abfd,
 
   versados_pass_2 (abfd);
   src = section->relocation;
+
   if (!EDATA (abfd, section->target_index).donerel)
     {
       EDATA (abfd, section->target_index).donerel = 1;
@@ -766,14 +818,21 @@ versados_canonicalize_reloc (bfd *abfd,
          int esdid = (int) (size_t) src[count].sym_ptr_ptr;
 
          if (esdid == 0)
-           src[count].sym_ptr_ptr = bfd_abs_section.symbol_ptr_ptr;
+           src[count].sym_ptr_ptr = bfd_abs_section_ptr->symbol_ptr_ptr;
          else if (esdid < ES_BASE)
            {
              /* Section relative thing.  */
              struct esdid *e = &EDATA (abfd, esdid - 1);
 
-             src[count].sym_ptr_ptr = e->section->symbol_ptr_ptr;
+             /* PR 17512: file:cd92277c.  */
+             if (e->section)
+               src[count].sym_ptr_ptr = e->section->symbol_ptr_ptr;
+             else
+               src[count].sym_ptr_ptr = bfd_und_section_ptr->symbol_ptr_ptr;
            }
+         /* PR 17512: file:3757-2936-0.004.  */
+         else if ((unsigned) (esdid - ES_BASE) >= bfd_get_symcount (abfd))
+           src[count].sym_ptr_ptr = bfd_und_section_ptr->symbol_ptr_ptr;
          else
            src[count].sym_ptr_ptr = symbols + esdid - ES_BASE;
        }
@@ -786,39 +845,44 @@ versados_canonicalize_reloc (bfd *abfd,
   return section->reloc_count;
 }
 
-#define        versados_close_and_cleanup                    _bfd_generic_close_and_cleanup
-#define versados_bfd_free_cached_info                 _bfd_generic_bfd_free_cached_info
-#define versados_new_section_hook                     _bfd_generic_new_section_hook
-#define versados_bfd_is_target_special_symbol   ((bfd_boolean (*) (bfd *, asymbol *)) bfd_false)
-#define versados_bfd_is_local_label_name              bfd_generic_is_local_label_name
-#define versados_get_lineno                           _bfd_nosymbols_get_lineno
-#define versados_find_nearest_line                    _bfd_nosymbols_find_nearest_line
-#define versados_find_inliner_info                    _bfd_nosymbols_find_inliner_info
-#define versados_make_empty_symbol                    _bfd_generic_make_empty_symbol
-#define versados_bfd_make_debug_symbol                _bfd_nosymbols_bfd_make_debug_symbol
-#define versados_read_minisymbols                     _bfd_generic_read_minisymbols
-#define versados_minisymbol_to_symbol                 _bfd_generic_minisymbol_to_symbol
-#define versados_bfd_reloc_type_lookup                _bfd_norelocs_bfd_reloc_type_lookup
-#define versados_bfd_reloc_name_lookup          _bfd_norelocs_bfd_reloc_name_lookup
-#define versados_set_arch_mach                        bfd_default_set_arch_mach
+#define        versados_close_and_cleanup                    _bfd_generic_close_and_cleanup
+#define versados_bfd_free_cached_info                _bfd_generic_bfd_free_cached_info
+#define versados_new_section_hook                    _bfd_generic_new_section_hook
+#define versados_bfd_is_target_special_symbol  ((bfd_boolean (*) (bfd *, asymbol *)) bfd_false)
+#define versados_bfd_is_local_label_name             bfd_generic_is_local_label_name
+#define versados_get_lineno                          _bfd_nosymbols_get_lineno
+#define versados_find_nearest_line                   _bfd_nosymbols_find_nearest_line
+#define versados_find_line                           _bfd_nosymbols_find_line
+#define versados_find_inliner_info                   _bfd_nosymbols_find_inliner_info
+#define versados_get_symbol_version_string           _bfd_nosymbols_get_symbol_version_string
+#define versados_make_empty_symbol                   _bfd_generic_make_empty_symbol
+#define versados_bfd_make_debug_symbol               _bfd_nosymbols_bfd_make_debug_symbol
+#define versados_read_minisymbols                    _bfd_generic_read_minisymbols
+#define versados_minisymbol_to_symbol                _bfd_generic_minisymbol_to_symbol
+#define versados_bfd_reloc_type_lookup               _bfd_norelocs_bfd_reloc_type_lookup
+#define versados_bfd_reloc_name_lookup         _bfd_norelocs_bfd_reloc_name_lookup
+#define versados_set_arch_mach                       bfd_default_set_arch_mach
 #define versados_bfd_get_relocated_section_contents   bfd_generic_get_relocated_section_contents
-#define versados_bfd_relax_section                    bfd_generic_relax_section
-#define versados_bfd_gc_sections                      bfd_generic_gc_sections
-#define versados_bfd_merge_sections                   bfd_generic_merge_sections
-#define versados_bfd_is_group_section                 bfd_generic_is_group_section
-#define versados_bfd_discard_group                    bfd_generic_discard_group
-#define versados_section_already_linked               _bfd_generic_section_already_linked
-#define versados_bfd_define_common_symbol             bfd_generic_define_common_symbol
-#define versados_bfd_link_hash_table_create           _bfd_generic_link_hash_table_create
-#define versados_bfd_link_hash_table_free             _bfd_generic_link_hash_table_free
-#define versados_bfd_link_add_symbols                 _bfd_generic_link_add_symbols
-#define versados_bfd_link_just_syms                   _bfd_generic_link_just_syms
+#define versados_bfd_relax_section                   bfd_generic_relax_section
+#define versados_bfd_gc_sections                     bfd_generic_gc_sections
+#define versados_bfd_lookup_section_flags            bfd_generic_lookup_section_flags
+#define versados_bfd_merge_sections                  bfd_generic_merge_sections
+#define versados_bfd_is_group_section                bfd_generic_is_group_section
+#define versados_bfd_discard_group                   bfd_generic_discard_group
+#define versados_section_already_linked                      _bfd_generic_section_already_linked
+#define versados_bfd_define_common_symbol            bfd_generic_define_common_symbol
+#define versados_bfd_define_start_stop               bfd_generic_define_start_stop
+#define versados_bfd_link_hash_table_create          _bfd_generic_link_hash_table_create
+#define versados_bfd_link_add_symbols                _bfd_generic_link_add_symbols
+#define versados_bfd_link_just_syms                  _bfd_generic_link_just_syms
 #define versados_bfd_copy_link_hash_symbol_type \
   _bfd_generic_copy_link_hash_symbol_type
-#define versados_bfd_final_link                       _bfd_generic_final_link
-#define versados_bfd_link_split_section               _bfd_generic_link_split_section
+#define versados_bfd_final_link                              _bfd_generic_final_link
+#define versados_bfd_link_split_section                      _bfd_generic_link_split_section
+#define versados_bfd_link_check_relocs               _bfd_generic_link_check_relocs
+#define versados_set_reloc                           _bfd_generic_set_reloc
 
-const bfd_target versados_vec =
+const bfd_target m68k_versados_vec =
 {
   "versados",                  /* Name.  */
   bfd_target_versados_flavour,
@@ -832,6 +896,7 @@ const bfd_target versados_vec =
   0,                           /* Leading underscore.  */
   ' ',                         /* AR_pad_char.  */
   16,                          /* AR_max_namelen.  */
+  0,                           /* match priority.  */
   bfd_getb64, bfd_getb_signed_64, bfd_putb64,
   bfd_getb32, bfd_getb_signed_32, bfd_putb32,
   bfd_getb16, bfd_getb_signed_16, bfd_putb16,  /* Data.  */
This page took 0.029974 seconds and 4 git commands to generate.