Do not allow records to cross a 64K boundary - wrap if necessary.
[deliverable/binutils-gdb.git] / bfd / ihex.c
index 512bf475979fae145eb35e92e2502b1467d058ef..613eaf3b189fdd75aacbb049699f11178884e353 100644 (file)
@@ -1,5 +1,6 @@
 /* BFD back-end for Intel Hex objects.
-   Copyright 1995 Free Software Foundation, Inc.
+   Copyright 1995, 1996, 1998, 1999, 2000, 2001, 2002
+   Free Software Foundation, Inc.
    Written by Ian Lance Taylor of Cygnus Support <ian@cygnus.com>.
 
 This file is part of BFD, the Binary File Descriptor library.
@@ -104,14 +105,25 @@ Another document reports these additional types:
    10..13      Upper 16 bits of start address
    14..15      Checksum in hex notation
    16..17      Carriage return, line feed
+
+The MRI compiler uses this, which is a repeat of type 5:
+
+  EXTENDED START RECORD
+   Byte 1      Header = colon (:)
+   2..3                The byte count, must be "04"
+   4..7                Load address, must be "0000"
+   8..9                Record type, must be "05"
+   10..13      Upper 16 bits of start address
+   14..17      Lower 16 bits of start address
+   18..19      Checksum in hex notation
+   20..21      Carriage return, line feed
 */
 
 #include "bfd.h"
 #include "sysdep.h"
 #include "libbfd.h"
 #include "libiberty.h"
-
-#include <ctype.h>
+#include "safe-ctype.h"
 
 static void ihex_init PARAMS ((void));
 static boolean ihex_mkobject PARAMS ((bfd *));
@@ -125,15 +137,15 @@ static boolean ihex_get_section_contents
 static boolean ihex_set_section_contents
   PARAMS ((bfd *, asection *, PTR, file_ptr, bfd_size_type));
 static boolean ihex_write_record
-  PARAMS ((bfd *, bfd_size_type, bfd_vma, unsigned int, bfd_byte *));
+  PARAMS ((bfd *, size_t, unsigned int, unsigned int, bfd_byte *));
 static boolean ihex_write_object_contents PARAMS ((bfd *));
-static asymbol *ihex_make_empty_symbol PARAMS ((bfd *));
 static boolean ihex_set_arch_mach
   PARAMS ((bfd *, enum bfd_architecture, unsigned long));
+static int ihex_sizeof_headers PARAMS ((bfd *, boolean));
 
 /* The number of bytes we put on one line during output.  */
 
-#define CHUNK (21)
+#define CHUNK 16
 
 /* Macros for converting between hex and binary. */
 
@@ -184,9 +196,9 @@ ihex_mkobject (abfd)
   if (abfd->tdata.ihex_data == NULL)
     {
       struct ihex_data_struct *tdata;
+      bfd_size_type amt = sizeof (struct ihex_data_struct);
 
-      tdata = ((struct ihex_data_struct *)
-              bfd_alloc (abfd, sizeof (struct ihex_data_struct)));
+      tdata = (struct ihex_data_struct *) bfd_alloc (abfd, amt);
       if (tdata == NULL)
        return false;
       abfd->tdata.ihex_data = tdata;
@@ -207,7 +219,7 @@ ihex_get_byte (abfd, errorptr)
 {
   bfd_byte c;
 
-  if (bfd_read (&c, 1, 1, abfd) != 1)
+  if (bfd_bread (&c, (bfd_size_type) 1, abfd) != 1)
     {
       if (bfd_get_error () != bfd_error_file_truncated)
        *errorptr = true;
@@ -235,7 +247,7 @@ ihex_bad_byte (abfd, lineno, c, error)
     {
       char buf[10];
 
-      if (! isprint (c))
+      if (! ISPRINT (c))
        sprintf (buf, "\\%03o", (unsigned int) c);
       else
        {
@@ -243,8 +255,8 @@ ihex_bad_byte (abfd, lineno, c, error)
          buf[1] = '\0';
        }
       (*_bfd_error_handler)
-       ("%s:%d: unexpected character `%s' in Intel Hex file\n",
-        bfd_get_filename (abfd), lineno, buf);
+       (_("%s:%d: unexpected character `%s' in Intel Hex file\n"),
+        bfd_archive_filename (abfd), lineno, buf);
       bfd_set_error (bfd_error_bad_value);
     }
 }
@@ -259,9 +271,9 @@ ihex_scan (abfd)
   bfd_vma segbase;
   bfd_vma extbase;
   asection *sec;
-  int lineno;
+  unsigned int lineno;
   boolean error;
-  bfd_byte *buf;
+  bfd_byte *buf = NULL;
   size_t bufsize;
   int c;
 
@@ -270,12 +282,11 @@ ihex_scan (abfd)
 
   abfd->start_address = 0;
 
-  extbase = 0;
   segbase = 0;
+  extbase = 0;
   sec = NULL;
   lineno = 1;
   error = false;
-  buf = NULL;
   bufsize = 0;
   while ((c = ihex_get_byte (abfd, &error)) != EOF)
     {
@@ -308,7 +319,7 @@ ihex_scan (abfd)
 
          /* Read the header bytes.  */
 
-         if (bfd_read (hdr, 1, 8, abfd) != 8)
+         if (bfd_bread (hdr, (bfd_size_type) 8, abfd) != 8)
            goto error_return;
 
          for (i = 0; i < 8; i++)
@@ -329,13 +340,13 @@ ihex_scan (abfd)
          chars = len * 2 + 2;
          if (chars >= bufsize)
            {
-             buf = (bfd_byte *) bfd_realloc (buf, chars);
+             buf = (bfd_byte *) bfd_realloc (buf, (bfd_size_type) chars);
              if (buf == NULL)
                goto error_return;
              bufsize = chars;
            }
 
-         if (bfd_read (buf, 1, chars, abfd) != chars)
+         if (bfd_bread (buf, (bfd_size_type) chars, abfd) != chars)
            goto error_return;
 
          for (i = 0; i < chars; i++)
@@ -354,8 +365,8 @@ ihex_scan (abfd)
          if (((- chksum) & 0xff) != (unsigned int) HEX2 (buf + 2 * i))
            {
              (*_bfd_error_handler)
-               ("%s:%d: bad checksum in Intel Hex file (expected %u, found %u)",
-                bfd_get_filename (abfd), lineno,
+               (_("%s:%u: bad checksum in Intel Hex file (expected %u, found %u)"),
+                bfd_archive_filename (abfd), lineno,
                 (- chksum) & 0xff, (unsigned int) HEX2 (buf + 2 * i));
              bfd_set_error (bfd_error_bad_value);
              goto error_return;
@@ -376,9 +387,11 @@ ihex_scan (abfd)
                {
                  char secbuf[20];
                  char *secname;
+                 bfd_size_type amt;
 
                  sprintf (secbuf, ".sec%d", bfd_count_sections (abfd) + 1);
-                 secname = (char *) bfd_alloc (abfd, strlen (secbuf) + 1);
+                 amt = strlen (secbuf) + 1;
+                 secname = (char *) bfd_alloc (abfd, amt);
                  if (secname == NULL)
                    goto error_return;
                  strcpy (secname, secbuf);
@@ -406,8 +419,8 @@ ihex_scan (abfd)
              if (len != 2)
                {
                  (*_bfd_error_handler)
-                   ("%s:%d: bad extended address record length in Intel Hex file",
-                    bfd_get_filename (abfd), lineno);
+                   (_("%s:%u: bad extended address record length in Intel Hex file"),
+                    bfd_archive_filename (abfd), lineno);
                  bfd_set_error (bfd_error_bad_value);
                  goto error_return;
                }
@@ -423,8 +436,8 @@ ihex_scan (abfd)
              if (len != 4)
                {
                  (*_bfd_error_handler)
-                   ("%s:%d: bad extended start address length in Intel Hex file",
-                    bfd_get_filename (abfd), lineno);
+                   (_("%s:%u: bad extended start address length in Intel Hex file"),
+                    bfd_archive_filename (abfd), lineno);
                  bfd_set_error (bfd_error_bad_value);
                  goto error_return;
                }
@@ -440,13 +453,13 @@ ihex_scan (abfd)
              if (len != 2)
                {
                  (*_bfd_error_handler)
-                   ("%s:%d: bad extended linear address record length in Intel Hex file",
-                    bfd_get_filename (abfd), lineno);
+                   (_("%s:%u: bad extended linear address record length in Intel Hex file"),
+                    bfd_archive_filename (abfd), lineno);
                  bfd_set_error (bfd_error_bad_value);
                  goto error_return;
                }
 
-             segbase = HEX4 (buf) << 16;
+             extbase = HEX4 (buf) << 16;
 
              sec = NULL;
 
@@ -454,16 +467,19 @@ ihex_scan (abfd)
 
            case 5:
              /* An extended linear start address record.  */
-             if (len != 2)
+             if (len != 2 && len != 4)
                {
                  (*_bfd_error_handler)
-                   ("%s:%d: bad extended linear start address length in Intel Hex file",
-                    bfd_get_filename (abfd), lineno);
+                   (_("%s:%u: bad extended linear start address length in Intel Hex file"),
+                    bfd_archive_filename (abfd), lineno);
                  bfd_set_error (bfd_error_bad_value);
                  goto error_return;
                }
 
-             abfd->start_address += HEX4 (buf) << 16;
+             if (len == 2)
+               abfd->start_address += HEX4 (buf) << 16;
+             else
+               abfd->start_address = (HEX4 (buf) << 16) + HEX4 (buf + 4);
 
              sec = NULL;
 
@@ -471,8 +487,8 @@ ihex_scan (abfd)
 
            default:
              (*_bfd_error_handler)
-               ("%s:%d: unrecognized ihex type %u in Intel Hex file\n",
-                bfd_get_filename (abfd), lineno, type);
+               (_("%s:%u: unrecognized ihex type %u in Intel Hex file\n"),
+                bfd_archive_filename (abfd), lineno, type);
              bfd_set_error (bfd_error_bad_value);
              goto error_return;
            }
@@ -507,7 +523,7 @@ ihex_object_p (abfd)
 
   if (bfd_seek (abfd, (file_ptr) 0, SEEK_SET) != 0)
     return NULL;
-  if (bfd_read (b, 1, 9, abfd) != 9)
+  if (bfd_bread (b, (bfd_size_type) 9, abfd) != 9)
     {
       if (bfd_get_error () == bfd_error_file_truncated)
        bfd_set_error (bfd_error_wrong_format);
@@ -555,7 +571,7 @@ ihex_read_section (abfd, section, contents)
 {
   int c;
   bfd_byte *p;
-  bfd_byte *buf;
+  bfd_byte *buf = NULL;
   size_t bufsize;
   boolean error;
 
@@ -563,7 +579,6 @@ ihex_read_section (abfd, section, contents)
     goto error_return;
 
   p = contents;
-  buf = NULL;
   bufsize = 0;
   error = false;
   while ((c = ihex_get_byte (abfd, &error)) != EOF)
@@ -581,7 +596,7 @@ ihex_read_section (abfd, section, contents)
          know the exact format.  */
       BFD_ASSERT (c == ':');
 
-      if (bfd_read (hdr, 1, 8, abfd) != 8)
+      if (bfd_bread (hdr, (bfd_size_type) 8, abfd) != 8)
        goto error_return;
 
       len = HEX2 (hdr);
@@ -592,21 +607,21 @@ ihex_read_section (abfd, section, contents)
       if (type != 0)
        {
          (*_bfd_error_handler)
-           ("%s: internal error in ihex_read_section",
-            bfd_get_filename (abfd));
+           (_("%s: internal error in ihex_read_section"),
+            bfd_archive_filename (abfd));
          bfd_set_error (bfd_error_bad_value);
          goto error_return;
        }
 
       if (len * 2 > bufsize)
        {
-         buf = (bfd_byte *) bfd_realloc (buf, len * 2);
+         buf = (bfd_byte *) bfd_realloc (buf, (bfd_size_type) len * 2);
          if (buf == NULL)
            goto error_return;
          bufsize = len * 2;
        }
 
-      if (bfd_read (buf, 1, len * 2, abfd) != len * 2)
+      if (bfd_bread (buf, (bfd_size_type) len * 2, abfd) != len * 2)
        goto error_return;
 
       for (i = 0; i < len; i++)
@@ -620,15 +635,15 @@ ihex_read_section (abfd, section, contents)
        }
 
       /* Skip the checksum.  */
-      if (bfd_read (buf, 1, 2, abfd) != 2)
+      if (bfd_bread (buf, (bfd_size_type) 2, abfd) != 2)
        goto error_return;
     }
 
   if ((bfd_size_type) (p - contents) < section->_raw_size)
     {
       (*_bfd_error_handler)
-       ("%s: bad section length in ihex_read_section",
-        bfd_get_filename (abfd));
+       (_("%s: bad section length in ihex_read_section"),
+        bfd_archive_filename (abfd));
       bfd_set_error (bfd_error_bad_value);
       goto error_return;
     }
@@ -682,14 +697,15 @@ ihex_set_section_contents (abfd, section, location, offset, count)
   struct ihex_data_list *n;
   bfd_byte *data;
   struct ihex_data_struct *tdata;
+  bfd_size_type amt;
 
   if (count == 0
       || (section->flags & SEC_ALLOC) == 0
       || (section->flags & SEC_LOAD) == 0)
     return true;
 
-  n = ((struct ihex_data_list *)
-       bfd_alloc (abfd, sizeof (struct ihex_data_list)));
+  amt = sizeof (struct ihex_data_list);
+  n = (struct ihex_data_list *) bfd_alloc (abfd, amt);
   if (n == NULL)
     return false;
 
@@ -734,8 +750,8 @@ ihex_set_section_contents (abfd, section, location, offset, count)
 static boolean
 ihex_write_record (abfd, count, addr, type, data)
      bfd *abfd;
-     bfd_size_type count;
-     bfd_vma addr;
+     size_t count;
+     unsigned int addr;
      unsigned int type;
      bfd_byte *data;
 {
@@ -744,6 +760,7 @@ ihex_write_record (abfd, count, addr, type, data)
   char *p;
   unsigned int chksum;
   unsigned int i;
+  size_t total;
 
 #define TOHEX(buf, v) \
   ((buf)[0] = digs[((v) >> 4) & 0xf], (buf)[1] = digs[(v) & 0xf])
@@ -766,7 +783,8 @@ ihex_write_record (abfd, count, addr, type, data)
   p[2] = '\r';
   p[3] = '\n';
 
-  if (bfd_write (buf, 1, 9 + count * 2 + 4, abfd) != 9 + count * 2 + 4)
+  total = 9 + count * 2 + 4;
+  if (bfd_bwrite (buf, (bfd_size_type) total, abfd) != total)
     return false;
 
   return true;
@@ -778,12 +796,12 @@ static boolean
 ihex_write_object_contents (abfd)
      bfd *abfd;
 {
-  bfd_vma extbase;
   bfd_vma segbase;
+  bfd_vma extbase;
   struct ihex_data_list *l;
 
-  extbase = 0;
   segbase = 0;
+  extbase = 0;
   for (l = abfd->tdata.ihex_data->head; l != NULL; l = l->next)
     {
       bfd_vma where;
@@ -795,27 +813,46 @@ ihex_write_object_contents (abfd)
       count = l->size;
       while (count > 0)
        {
-         bfd_size_type now;
+         size_t now;
+         unsigned int rec_addr;
 
          now = count;
-         if (now > CHUNK)
+         if (count > CHUNK)
            now = CHUNK;
 
-         if (where > extbase + segbase + 0xffff)
+         if (where > segbase + extbase + 0xffff)
            {
              bfd_byte addr[2];
 
              /* We need a new base address.  */
              if (where <= 0xfffff)
                {
+                 /* The addresses should be sorted.  */
+                 BFD_ASSERT (extbase == 0);
+
                  segbase = where & 0xf0000;
-                 addr[0] = (segbase >> 12) & 0xff;
-                 addr[1] = (segbase >> 4) & 0xff;
+                 addr[0] = (bfd_byte)(segbase >> 12) & 0xff;
+                 addr[1] = (bfd_byte)(segbase >> 4) & 0xff;
                  if (! ihex_write_record (abfd, 2, 0, 2, addr))
                    return false;
                }
              else
                {
+                 /* The extended address record and the extended
+                     linear address record are combined, at least by
+                     some readers.  We need an extended linear address
+                     record here, so if we've already written out an
+                     extended address record, zero it out to avoid
+                     confusion.  */
+                 if (segbase != 0)
+                   {
+                     addr[0] = 0;
+                     addr[1] = 0;
+                     if (! ihex_write_record (abfd, 2, 0, 2, addr))
+                       return false;
+                     segbase = 0;
+                   }
+
                  extbase = where & 0xffff0000;
                  if (where > extbase + 0xffff)
                    {
@@ -823,20 +860,25 @@ ihex_write_object_contents (abfd)
 
                      sprintf_vma (buf, where);
                      (*_bfd_error_handler)
-                       ("%s: address 0x%s out of range for Intex Hex file",
+                       (_("%s: address 0x%s out of range for Intel Hex file"),
                         bfd_get_filename (abfd), buf);
                      bfd_set_error (bfd_error_bad_value);
                      return false;
                    }
-                 addr[0] = (extbase >> 24) & 0xff;
-                 addr[1] = (extbase >> 16) & 0xff;
+                 addr[0] = (bfd_byte)(extbase >> 24) & 0xff;
+                 addr[1] = (bfd_byte)(extbase >> 16) & 0xff;
                  if (! ihex_write_record (abfd, 2, 0, 4, addr))
                    return false;
                }
            }
 
-         if (! ihex_write_record (abfd, now, where - (extbase + segbase),
-                                  0, p))
+         rec_addr = where - (extbase + segbase);
+
+          /* Output records shouldn't cross 64K boundaries.  */
+          if (rec_addr + now > 0xfffff)
+            now = 0xffff - rec_addr;
+
+         if (! ihex_write_record (abfd, now, rec_addr, 0, p))
            return false;
 
          where += now;
@@ -852,21 +894,24 @@ ihex_write_object_contents (abfd)
 
       start = abfd->start_address;
 
-      if (start > 0xfffff)
+      if (start <= 0xfffff)
        {
-         startbuf[0] = (start >> 24) & 0xff;
-         startbuf[1] = (start >> 16) & 0xff;
-         if (! ihex_write_record (abfd, 2, 0, 5, startbuf))
+         startbuf[0] = (bfd_byte)((start & 0xf0000) >> 12) & 0xff;
+         startbuf[1] = 0;
+         startbuf[2] = (bfd_byte)(start >> 8) & 0xff;
+         startbuf[3] = (bfd_byte)start & 0xff;
+         if (! ihex_write_record (abfd, 4, 0, 3, startbuf))
+           return false;
+       }
+      else
+       {
+         startbuf[0] = (bfd_byte)(start >> 24) & 0xff;
+         startbuf[1] = (bfd_byte)(start >> 16) & 0xff;
+         startbuf[2] = (bfd_byte)(start >> 8) & 0xff;
+         startbuf[3] = (bfd_byte)start & 0xff;
+         if (! ihex_write_record (abfd, 4, 0, 5, startbuf))
            return false;
-         start &= 0xffff;
        }
-
-      startbuf[0] = ((start & 0xf0000) >> 12) & 0xff;
-      startbuf[1] = 0;
-      startbuf[2] = (start >> 8) & 0xff;
-      startbuf[3] = start & 0xff;
-      if (! ihex_write_record (abfd, 4, 0, 3, startbuf))
-       return false;
     }
 
   if (! ihex_write_record (abfd, 0, 0, 1, NULL))
@@ -875,21 +920,6 @@ ihex_write_object_contents (abfd)
   return true;
 }
 
-/* Make an empty symbol.  This is required only because
-   bfd_make_section_anyway wants to create a symbol for the section.  */
-
-static asymbol *
-ihex_make_empty_symbol (abfd)
-     bfd *abfd;
-{
-  asymbol *new;
-
-  new = (asymbol *) bfd_zalloc (abfd, sizeof (asymbol));
-  if (new != NULL)
-    new->the_bfd = abfd;
-  return new;
-}
-
 /* Set the architecture for the output file.  The architecture is
    irrelevant, so we ignore errors about unknown architectures.  */
 
@@ -899,7 +929,11 @@ ihex_set_arch_mach (abfd, arch, mach)
      enum bfd_architecture arch;
      unsigned long mach;
 {
-  bfd_default_set_arch_mach (abfd, arch, mach);
+  if (! bfd_default_set_arch_mach (abfd, arch, mach))
+    {
+      if (arch != bfd_arch_unknown)
+       return false;
+    }
   return true;
 }
 
@@ -908,8 +942,8 @@ ihex_set_arch_mach (abfd, arch, mach)
 /*ARGSUSED*/
 static int
 ihex_sizeof_headers (abfd, exec)
-     bfd *abfd;
-     boolean exec;
+     bfd *abfd ATTRIBUTE_UNUSED;
+     boolean exec ATTRIBUTE_UNUSED;
 {
   return 0;
 }
@@ -922,11 +956,13 @@ ihex_sizeof_headers (abfd, exec)
 #define ihex_get_section_contents_in_window \
   _bfd_generic_get_section_contents_in_window
 
-#define ihex_get_symtab_upper_bound _bfd_nosymbols_get_symtab_upper_bound
-#define ihex_get_symtab _bfd_nosymbols_get_symtab
+#define ihex_get_symtab_upper_bound bfd_0l
+#define ihex_get_symtab \
+  ((long (*) PARAMS ((bfd *, asymbol **))) bfd_0l)
+#define ihex_make_empty_symbol _bfd_generic_make_empty_symbol
 #define ihex_print_symbol _bfd_nosymbols_print_symbol
 #define ihex_get_symbol_info _bfd_nosymbols_get_symbol_info
-#define ihex_bfd_is_local_label _bfd_nosymbols_bfd_is_local_label
+#define ihex_bfd_is_local_label_name _bfd_nosymbols_bfd_is_local_label_name
 #define ihex_get_lineno _bfd_nosymbols_get_lineno
 #define ihex_find_nearest_line _bfd_nosymbols_find_nearest_line
 #define ihex_bfd_make_debug_symbol _bfd_nosymbols_bfd_make_debug_symbol
@@ -942,6 +978,8 @@ ihex_sizeof_headers (abfd, exec)
 #define ihex_bfd_get_relocated_section_contents \
   bfd_generic_get_relocated_section_contents
 #define ihex_bfd_relax_section bfd_generic_relax_section
+#define ihex_bfd_gc_sections bfd_generic_gc_sections
+#define ihex_bfd_merge_sections bfd_generic_merge_sections
 #define ihex_bfd_link_hash_table_create _bfd_generic_link_hash_table_create
 #define ihex_bfd_link_add_symbols _bfd_generic_link_add_symbols
 #define ihex_bfd_final_link _bfd_generic_final_link
@@ -953,8 +991,8 @@ const bfd_target ihex_vec =
 {
   "ihex",                      /* name */
   bfd_target_ihex_flavour,
-  true,                                /* target byte order */
-  true,                                /* target headers byte order */
+  BFD_ENDIAN_UNKNOWN,          /* target byte order */
+  BFD_ENDIAN_UNKNOWN,          /* target headers byte order */
   0,                           /* object flags */
   (SEC_HAS_CONTENTS | SEC_ALLOC | SEC_LOAD),   /* section flags */
   0,                           /* leading underscore */
@@ -996,5 +1034,7 @@ const bfd_target ihex_vec =
   BFD_JUMP_TABLE_LINK (ihex),
   BFD_JUMP_TABLE_DYNAMIC (_bfd_nodynamic),
 
+  NULL,
+
   (PTR) 0
 };
This page took 0.044318 seconds and 4 git commands to generate.