Touches most files in bfd/, so likely will be blamed for everything..

[deliverable/binutils-gdb.git] / bfd / ihex.c

diff --git a/bfd/ihex.c b/bfd/ihex.c
index 700dfa98a18ed1bb7ae11f435300b379f820e225..83808ee89699d4fe2d5be834dd3f899bb4853029 100644 (file)
--- a/bfd/ihex.c
+++ b/bfd/ihex.c
@@ -1,5 +1,6 @@
 /* BFD back-end for Intel Hex objects.
-   Copyright 1995, 1996, 1998, 1999, 2000 Free Software Foundation, Inc.
+   Copyright 1995, 1996, 1998, 1999, 2000, 2001
+   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.
@@ -137,7 +138,7 @@ 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
@@ -197,9 +198,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;
@@ -220,7 +221,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;
@@ -272,7 +273,7 @@ ihex_scan (abfd)
   bfd_vma segbase;
   bfd_vma extbase;
   asection *sec;
-  int lineno;
+  unsigned int lineno;
   boolean error;
   bfd_byte *buf = NULL;
   size_t bufsize;
@@ -320,7 +321,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++)
@@ -341,13 +342,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++)
@@ -366,7 +367,7 @@ 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)"),
+               (_("%s:%u: bad checksum in Intel Hex file (expected %u, found %u)"),
                 bfd_get_filename (abfd), lineno,
                 (- chksum) & 0xff, (unsigned int) HEX2 (buf + 2 * i));
              bfd_set_error (bfd_error_bad_value);
@@ -388,9 +389,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);
@@ -418,7 +421,7 @@ ihex_scan (abfd)
              if (len != 2)
                {
                  (*_bfd_error_handler)
-                   (_("%s:%d: bad extended address record length in Intel Hex file"),
+                   (_("%s:%u: bad extended address record length in Intel Hex file"),
                     bfd_get_filename (abfd), lineno);
                  bfd_set_error (bfd_error_bad_value);
                  goto error_return;
@@ -435,7 +438,7 @@ ihex_scan (abfd)
              if (len != 4)
                {
                  (*_bfd_error_handler)
-                   (_("%s:%d: bad extended start address length in Intel Hex file"),
+                   (_("%s:%u: bad extended start address length in Intel Hex file"),
                     bfd_get_filename (abfd), lineno);
                  bfd_set_error (bfd_error_bad_value);
                  goto error_return;
@@ -452,7 +455,7 @@ ihex_scan (abfd)
              if (len != 2)
                {
                  (*_bfd_error_handler)
-                   (_("%s:%d: bad extended linear address record length in Intel Hex file"),
+                   (_("%s:%u: bad extended linear address record length in Intel Hex file"),
                     bfd_get_filename (abfd), lineno);
                  bfd_set_error (bfd_error_bad_value);
                  goto error_return;
@@ -469,7 +472,7 @@ ihex_scan (abfd)
              if (len != 2 && len != 4)
                {
                  (*_bfd_error_handler)
-                   (_("%s:%d: bad extended linear start address length in Intel Hex file"),
+                   (_("%s:%u: bad extended linear start address length in Intel Hex file"),
                     bfd_get_filename (abfd), lineno);
                  bfd_set_error (bfd_error_bad_value);
                  goto error_return;
@@ -486,7 +489,7 @@ ihex_scan (abfd)
 
            default:
              (*_bfd_error_handler)
-               (_("%s:%d: unrecognized ihex type %u in Intel Hex file\n"),
+               (_("%s:%u: unrecognized ihex type %u in Intel Hex file\n"),
                 bfd_get_filename (abfd), lineno, type);
              bfd_set_error (bfd_error_bad_value);
              goto error_return;
@@ -522,7 +525,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);
@@ -595,7 +598,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);
@@ -614,13 +617,13 @@ ihex_read_section (abfd, section, contents)
 
       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++)
@@ -634,7 +637,7 @@ 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;
     }
 
@@ -696,14 +699,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;
 
@@ -748,8 +752,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;
 {
@@ -758,6 +762,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])
@@ -780,7 +785,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;
@@ -809,10 +815,11 @@ 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 > segbase + extbase + 0xffff)
@@ -867,8 +874,8 @@ ihex_write_object_contents (abfd)
                }
            }
 
-         if (! ihex_write_record (abfd, now, where - (extbase + segbase),
-                                  0, p))
+         rec_addr = where - (extbase + segbase);
+         if (! ihex_write_record (abfd, now, rec_addr, 0, p))
            return false;
 
          where += now;
@@ -919,7 +926,7 @@ ihex_make_empty_symbol (abfd)
 {
   asymbol *new;
 
-  new = (asymbol *) bfd_zalloc (abfd, sizeof (asymbol));
+  new = (asymbol *) bfd_zalloc (abfd, (bfd_size_type) sizeof (asymbol));
   if (new != NULL)
     new->the_bfd = abfd;
   return new;
@@ -1039,6 +1046,6 @@ const bfd_target ihex_vec =
   BFD_JUMP_TABLE_DYNAMIC (_bfd_nodynamic),
 
   NULL,
-  
+
   (PTR) 0
 };