In article <20101103180747.ga26...@cs.hut.fi>, Antti Kantee <po...@cs.hut.fi> wrote: >On Wed Nov 03 2010 at 12:10:27 -0400, Christos Zoulas wrote: >> Index: src/tests/net/icmp/t_ping.c >> diff -u src/tests/net/icmp/t_ping.c:1.8 src/tests/net/icmp/t_ping.c:1.9 >> --- src/tests/net/icmp/t_ping.c:1.8 Thu Aug 26 13:24:14 2010 >> +++ src/tests/net/icmp/t_ping.c Wed Nov 3 12:10:24 2010 >> @@ -1,4 +1,4 @@ >> -/* $NetBSD: t_ping.c,v 1.8 2010/08/26 17:24:14 pooka Exp $ */ >> +/* $NetBSD: t_ping.c,v 1.9 2010/11/03 16:10:24 christos Exp $ */ >> >> /*- >> * Copyright (c) 2010 The NetBSD Foundation, Inc. >> @@ -129,7 +129,7 @@ >> } >> >> static int >> -doping(const char *target, int loops, int pktsize) >> +doping(const char *target, int loops, size_t pktsize) >> { >> char sndbuf[IP_MAXPACKET - sizeof(struct ip)]; >> char recvbuf[IP_MAXPACKET]; > >Why did you change that? It breaks the test on sparc64 since setsockopt() >expects an int *.
Fixed. christos