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

Brian Somers brian at
Mon Jan 2 00:51:23 PST 2006

brian       2006-01-02 08:51:21 UTC

  FreeBSD src repository

  Modified files:
    usr.bin/su           su.c 
  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

More information about the cvs-all mailing list