PR/36997 CVS commit: [netbsd-6] src/sbin/ping

["Soren Jacobsen" <snj%netbsd.org@localhost>]

The following reply was made to PR bin/36997; it has been noted by GNATS.

From: "Soren Jacobsen" <snj%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/36997 CVS commit: [netbsd-6] src/sbin/ping
Date: Tue, 11 Jul 2017 21:16:08 +0000

 Module Name:	src
 Committed By:	snj
 Date:		Tue Jul 11 21:16:07 UTC 2017
 
 Modified Files:
 	src/sbin/ping [netbsd-6]: ping.c
 
 Log Message:
 Pull up following revision(s) (requested by dholland in ticket #1424):
 	sbin/ping/ping.c: revision 1.113 via patch
 PR bin/36997 Zafer Aydogan: ping doesn't validate numeric inputs enough.
 Check for values between INT_MAX and LONG_MAX (if they're different)
 when using strtol to get an int. This applies to the -c and -l options;
 the other uses were already checked.
 Also limit the inter-packet interval given with -i to values that
 don't cause integer overflow calling poll() with milliseconds.
 Really large intervals (the number is read as floating point) can
 produce positive poll() values but negative integers when converted to
 struct timespec; this produces behavior akin to using -l at first and
 could be construed as a local DoS vulnerability.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.102.2.1 -r1.102.2.2 src/sbin/ping/ping.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.