diff mbox

Fix nss/tst-nss-test1.c format warning

Message ID alpine.DEB.2.10.1411262246440.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 a warning "tst-nss-test1.c:25:3:
warning: format '%ju' expects argument of type 'uintmax_t', but
argument 2 has type 'int' [-Wformat=]".  The argument is a difference
of two pointers, a signed quantity of type ptrdiff_t for which the
right format is %td; this patch makes this test use that format.

Tested for 32-bit x86.

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

	* nss/tst-nss-test1.c (do_test): Use %td printf format for pointer
	difference, not %ju.

Comments

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

Patch

diff --git a/nss/tst-nss-test1.c b/nss/tst-nss-test1.c
index 4e443d4..c5750e0 100644
--- a/nss/tst-nss-test1.c
+++ b/nss/tst-nss-test1.c
@@ -21,7 +21,7 @@  do_test (void)
     if (p->pw_uid != *np || strncmp (p->pw_name, "name", 4) != 0
 	|| atol (p->pw_name + 4) != *np)
       {
-	printf ("passwd entry %ju wrong (%s, %u)\n",
+	printf ("passwd entry %td wrong (%s, %u)\n",
 		np - pwdids, p->pw_name, p->pw_uid);
 	retval = 1;
 	break;