diff options
| author |   hugh <hugh@openbsd.org> | 2002-01-28 08:08:20 +0000 |
|---|---|---|
| committer | hugh <hugh@openbsd.org> | 2002-01-28 08:08:20 +0000 |
| commit | 1e7ad54ac4e1e36719c0b67fab538cba451d5bcd (patch) | |
| tree | ca5c712329420dcd418009932aaf9842772e6608 | |
| parent | Fix memory leaks in login_getcapnum() and login_getcapsize(). (diff) | |
| download | wireguard-openbsd-1e7ad54ac4e1e36719c0b67fab538cba451d5bcd.tar.xz wireguard-openbsd-1e7ad54ac4e1e36719c0b67fab538cba451d5bcd.zip | |
Request enough buffer space to fit the maximum size of a packet.
Problem noticed by Art, and fix checked by Markus and Niels.
| -rw-r--r-- | sbin/ping/ping.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/sbin/ping/ping.c b/sbin/ping/ping.c index f6cb8d7a128..98e627e25e7 100644 --- a/sbin/ping/ping.c +++ b/sbin/ping/ping.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ping.c,v 1.47 2001/11/05 07:39:17 mpech Exp $ */ +/* $OpenBSD: ping.c,v 1.48 2002/01/28 08:08:20 hugh Exp $ */ /* $NetBSD: ping.c,v 1.20 1995/08/11 22:37:58 cgd Exp $ */ /* @@ -47,7 +47,7 @@ static char copyright[] = #if 0 static char sccsid[] = "@(#)ping.c 8.1 (Berkeley) 6/5/93"; #else -static char rcsid[] = "$OpenBSD: ping.c,v 1.47 2001/11/05 07:39:17 mpech Exp $"; +static char rcsid[] = "$OpenBSD: ping.c,v 1.48 2002/01/28 08:08:20 hugh Exp $"; #endif #endif /* not lint */ @@ -170,8 +170,7 @@ quad_t tsumsq = 0; /* sum of all times squared, for std. dev. */ int reset_kerninfo; #endif -#define DEFAULT_BUFSPACE 60*1024 /* default read buffer size */ -int bufspace = DEFAULT_BUFSPACE; +int bufspace = IP_MAXPACKET; void fill __P((char *, char *)); void catcher(), prtsig(), finish(), summary(int); @@ -467,9 +466,9 @@ main(argc, argv) if ((bufspace -= 1024) <= 0) err(1, "Cannot set the receive buffer size"); } - if (bufspace < DEFAULT_BUFSPACE) + if (bufspace < IP_MAXPACKET) warnx("Could only allocate a receive buffer of %i bytes (default %i)", - bufspace, DEFAULT_BUFSPACE); + bufspace, IP_MAXPACKET); if (to->sin_family == AF_INET) (void)printf("PING %s (%s): %d data bytes\n", hostname, |