06-17-05 10:53 PM
David Schwartz wrote:
> "David Mathog" <mathog@caltech.edu> wrote in message
> news:d8v116$b29$1@naig.caltech.edu...
>
>
>
>
> I believe those two lines are to drop permissions after the port is
> opened. You should *DEIFNITELY* not remove security-sensitive code unless
> you are sure you understand what is doing and can prove it isn't needed.
Good point. I put those back in and added instead an
seteuid(0);
at the bottom of the loop over hosts. That also cured the original
problem and should be about as secure as the original single target
rsh.
I'm still not clear what the setuid(uid) was doing since the uid value
was read from the original process via a getuid(uid), and uid didn't
change in between.
Thanks,
David Mathog
mathog@caltech.edu
[ Post a follow-up to this message ]
|