Re: [9fans] APE exit()

["andrey mirtchovski" <mirtchovski@gmail.com>]

I think I figured that one out.

/sys/src/ape/lib/ap/plan9/_buf.c:66 sets _killmuxsid() to be called
during exit, then forks the process nodefile. this is all fine until
exit time, when there is a race: _killmuxsid is called and tries to
kill the child via kill(-pidgroup) but the child has already exited.
kill() calls setpgid() on itself trying to become the same group as
the child, but since the child has exited setpgid() fails (there is no
such group anymore). kill() will then happily proceed and send SIGTERM
to the original note group with which it was born, which unfortunately
is the same notegroup of the shell, hence killing them both.

a proposed fix for /sys/src/ape/lib/ap/plan9/kill.c:

49,54c49,51
< 		if(setpgid(mpid, -pid) > 0) {
< 			r = note(mpid, msg, "/proc/%d/notepg");
< 			setpgid(mpid, sid);
< 		} else {
< 			r = -1;
< 		}
---
> 		setpgid(mpid, -pid);
> 		r = note(mpid, msg, "/proc/%d/notepg");
> 		setpgid(mpid, sid);