* ar.c (print_contents, extract_file): Cast the return value

	of fwrite to size_t.

diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index 13d20eb818620d913c8627b4ad1140bd65548a59..5f99b21423100f9a44c4d2b88c64a55fb139f5b4 100644 (file)
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -1,3 +1,8 @@
+2007-01-25  Kazu Hirata  <kazu@codesourcery.com>
+
+       * ar.c (print_contents, extract_file): Cast the return value
+       of fwrite to size_t.
+
 2007-01-12  Alan Modra  <amodra@bigpond.net.au>
 
        * ar.c (open_inarch): Check fwrite return.  Use size_t.

diff --git a/binutils/ar.c b/binutils/ar.c
index 115224221e6ddea6ad396a5d0aeb04547c4e2c4b..1fbacf9101ad78e87ecdb1ab6112b0bec3ccb7d8 100644 (file)
--- a/binutils/ar.c
+++ b/binutils/ar.c
@@ -805,7 +805,11 @@ print_contents (bfd *abfd)
        /* xgettext:c-format */
        fatal (_("%s is not a valid archive"),
               bfd_get_filename (bfd_my_archive (abfd)));
-      if (fwrite (cbuf, 1, nread, stdout) != nread)
+
+      /* fwrite in mingw32 may return int instead of size_t. Cast the
+        return value to size_t to avoid comparison between signed and
+        unsigned values.  */
+      if ((size_t) fwrite (cbuf, 1, nread, stdout) != nread)
        fatal ("stdout: %s", strerror (errno));
       ncopied += tocopy;
     }
@@ -885,7 +889,11 @@ extract_file (bfd *abfd)
 
            output_file = ostream;
          }
-       if (fwrite (cbuf, 1, nread, ostream) != nread)
+
+       /* fwrite in mingw32 may return int instead of size_t. Cast
+          the return value to size_t to avoid comparison between
+          signed and unsigned values.  */
+       if ((size_t) fwrite (cbuf, 1, nread, ostream) != nread)
          fatal ("%s: %s", output_filename, strerror (errno));
        ncopied += tocopy;
       }