On Mon, 12 Aug 2002, Andrey A. Chernov wrote:

> On Tue, Aug 13, 2002 at 03:15:02 +1000, Bruce Evans wrote:
> >
> > Anyway, it has one blatant style bug (not 1TBS) and no explanation of
> > the bug, so it should not have been committed verbatim.  See another
> > reply for an analysis of the patch.
> According to commit guide, explanations are not commited, only what is
> changed. For explanation (terminal group mismatch) read whole thread,
> especially David's analysis.

The analysis in the thread seems to be incomplete.

> What another reply you mean?

I just wrote one :-).

Rereading earlier analysis just gave me a hint about tcsetpgrp() to
the same value causes problems: it causes an unexpected SIGTTOU because
a background process attemps to do something (TIOCSPGRP) that can only
be done in the foreground (the background process is forced to wait
until it gets to the foreground, and it doesn't help for the change
to be null since the process must be in the foreground to even check
for null changes).  Presumably the child su is in the foreground at
that point.


To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message

Reply via email to