Mercurial > dropbear
diff svr-chansession.c @ 96:b8e36c1a5520
Mention the race condition between setting the childpid entry, and
the child exiting.
author | Matt Johnston <matt@ucc.asn.au> |
---|---|
date | Wed, 18 Aug 2004 16:42:08 +0000 |
parents | 6539c9480b23 |
children | 10f4d3319780 |
line wrap: on
line diff
--- a/svr-chansession.c Wed Aug 18 16:31:05 2004 +0000 +++ b/svr-chansession.c Wed Aug 18 16:42:08 2004 +0000 @@ -78,7 +78,6 @@ TRACE(("enter sigchld handler")); while ((pid = waitpid(-1, &status, WNOHANG)) > 0) { - /* find the corresponding chansess */ for (i = 0; i < svr_ses.childpidsize; i++) { if (svr_ses.childpids[i].pid == pid) { @@ -630,7 +629,10 @@ TRACE(("continue noptycommand: parent")); chansess->pid = pid; - /* add a child pid */ + /* add a child pid - Beware: there's a race between this, and the + * exec() called from the child. If the child finishes before we've + * done this (ie if it was a shell builtin and fast), we won't return a + * proper return code. For now, we ignore this case. */ addchildpid(chansess, pid); close(infds[FDIN]);