Subject: PR/28741 CVS commit: [netbsd-2] src/sbin/ping
To: None <gnats-admin@netbsd.org, netbsd-bugs@netbsd.org, mike@ethmoid.org>
From: Manuel Bouyer <bouyer@netbsd.org>
List: netbsd-bugs
Date: 08/11/2007 14:15:07
The following reply was made to PR bin/28741; it has been noted by GNATS.
From: Manuel Bouyer <bouyer@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc:
Subject: PR/28741 CVS commit: [netbsd-2] src/sbin/ping
Date: Sat, 11 Aug 2007 14:10:49 +0000 (UTC)
Module Name: src
Committed By: bouyer
Date: Sat Aug 11 14:10:48 UTC 2007
Modified Files:
src/sbin/ping [netbsd-2]: ping.c
Log Message:
Pull up following revision(s) (requested by ghen in ticket #11352):
sbin/ping/ping.c: revision 1.83 - 1.85
PR/28741: Michael Santos: ping does [not] drop root privileges
Moved socket calls way up, and called seteuid(getuid()).
On 2nd thought, setuid() is better here.
Change error message to say "setuid", forgotten in previous commit.
Pointed out by Ray Lai, thanks!
To generate a diff of this commit:
cvs rdiff -r1.75.4.1 -r1.75.4.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.