Date: Sat, 11 Jul 1998 11:14:56 +1000 From: Bruce Evans <bde@zeta.org.au> To: bde@zeta.org.au, ghelmer@scl.ameslab.gov Cc: cvs-committers@FreeBSD.ORG, imp@village.org, roberto@keltia.freenix.fr Subject: Re: cvs commit: src/usr.bin/msgs msgs.c Message-ID: <199807110114.LAA00069@godzilla.zeta.org.au>
next in thread | raw e-mail | index | archive | help
>So, as I understand it now, the change > > seteuid(uid = getuid()); > setuid(uid); > >should be reverted to the original > > setuid(uid = getuid()); > >It seemed like an innocuous change. Thanks for catching this and educating >me. Well, it is innocuous, except it reduces portabilty (seteuid() isn't in POSIX.1). Here is an attempt at a portable version: uid = getuid(); (void)setuid(uid); (void)setuid(uid); I think this works under OpenBSD. It is obviously innocuous, but not obviously correct. Bruce To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199807110114.LAA00069>