Re: CVS commit: src/usr.bin/rsh

2014-10-30 Thread David Holland
On Thu, Oct 30, 2014 at 04:40:57PM +, David Holland wrote: > > } > > } Module Name:src > > } Committed By: dholland > > } Date: Thu Oct 30 06:13:50 UTC 2014 > > } > > } Modified Files: > > } src/usr.bin/rsh: rsh.c > > } > > } Log Message: > > } Dro

Re: CVS commit: src/usr.bin/rsh

2014-10-30 Thread David Holland
On Wed, Oct 29, 2014 at 11:54:41PM -0700, John Nemeth wrote: > On Oct 30, 6:13am, "David A. Holland" wrote: > } > } Module Name: src > } Committed By: dholland > } Date: Thu Oct 30 06:13:50 UTC 2014 > } > } Modified Files: > }src/usr.bin/rsh: rsh.c > } > }

Re: CVS commit: src/usr.bin/rsh

2014-10-30 Thread John Nemeth
On Oct 30, 6:13am, "David A. Holland" wrote: } } Module Name: src } Committed By: dholland } Date: Thu Oct 30 06:13:50 UTC 2014 } } Modified Files: } src/usr.bin/rsh: rsh.c } } Log Message: } Drop setuid before execing rlogin. Failure to do so should be } harmless, but is sloppy.

re: CVS commit: src/usr.bin/rsh

2014-10-30 Thread matthew green
"David A. Holland" writes: > Module Name: src > Committed By: dholland > Date: Thu Oct 30 06:13:50 UTC 2014 > > Modified Files: > src/usr.bin/rsh: rsh.c > > Log Message: > Drop setuid before execing rlogin. Failure to do so should be > harmless, but is sloppy. as jnemeth reminded