Date: Mon, 2 Jan 2006 08:51:21 +0000 (UTC) From: Brian Somers <brian@FreeBSD.org> To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/usr.bin/su su.c Message-ID: <200601020851.k028pMLR051063@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
brian 2006-01-02 08:51:21 UTC
FreeBSD src repository
Modified files:
usr.bin/su su.c
Log:
Remove broken code that mucks about with tcsetpgrp() -- even if
su isn't the foreground process. Hopefully this won't break PAM,
but I couldn't find any useful information about ache's theory
that it will.
Specifically, this change fixes the following:
# sh
# echo $$
# su - root -c id &
# echo $$
The PID output changes as su seems to be kill -STOP'ing itself
and catching the parent shell in the process. This is especially
bad if you add a ``su - user -c command &'' to an rc script!
Sponsored by: Sophos/Activestate
Not objected to by: des
Revision Changes Path
1.77 +0 -7 src/usr.bin/su/su.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200601020851.k028pMLR051063>