cvs commit: src/usr.bin/su su.c

[David Xu]

Brian Somers wrote:
> 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
> 
> 

Oh, you obviously have not checked history (discussion between bde@ and
me), please try following code:

%sh
$ su
apple# kill -STOP $$



^C^C^C