diff mbox

Fix stdio-common/tst-fmemopen.c format warnings

Message ID alpine.DEB.2.10.1411262247350.29828@digraph.polyomino.org.uk
State Committed
Headers

Commit Message

Joseph Myers Nov. 26, 2014, 10:47 p.m. UTC 
  Testing for 32-bit x86 shows up warnings in
stdio-common/tst-fmemopen.c where off_t values are passed to %zu
printf formats.  Since the values are in messages relating to function
calls where the relevant argument is of type size_t, it seems most
appropriate to cast explicitly to size_t when passing to printf, which
this patch does.

Tested for 32-bit x86.

2014-11-26  Joseph Myers  <joseph@codesourcery.com>

	* stdio-common/tst-fmemopen.c (do_test): Cast st_size values to
	size_t for %zu format.

Comments

Paul Eggert Nov. 27, 2014, 12:45 a.m. UTC | #1 
Thanks, this looks good.
diff mbox

Patch

diff --git a/stdio-common/tst-fmemopen.c b/stdio-common/tst-fmemopen.c
index aba9310..c5015a6 100644
--- a/stdio-common/tst-fmemopen.c
+++ b/stdio-common/tst-fmemopen.c
@@ -108,13 +108,13 @@  do_test (void)
 				  MAP_SHARED, fd, 0)) == MAP_FAILED)
     {
       printf ("mmap (NULL, %zu, PROT_READ, MAP_SHARED, %i, 0) failed\n",
-	      fs.st_size, fd);
+	      (size_t) fs.st_size, fd);
       return 5;
     }
 
   if ((fp = fmemopen (mmap_data, fs.st_size, "r")) == NULL)
     {
-      printf ("fmemopen (%p, %zu) failed\n", mmap_data, fs.st_size);
+      printf ("fmemopen (%p, %zu) failed\n", mmap_data, (size_t) fs.st_size);
       return 1;
     }