On 06/22/2007 04:43 PM, Joe Orton wrote: > On Thu, Jun 21, 2007 at 11:22:09PM +0200, Ruediger Pluem wrote: > >>On 06/21/2007 06:51 PM, Joe Orton wrote: >> >>>Secondly: I think this approach is unnecessarily complex. I think it's >>>sufficient to simply check whether the target process is in the right >>>process group before sending a signal, i.e. getpgid(pid) == getpgrp(). >>>This ensures that the parent will only kill things it created. >>> >>>It is reasonable to assume that the parent's process group holds exactly >>>the set of processes which is safe to kill - prefork relies on that >>>being so when handling SIGHUP already. >>> >>>Patch below is PoC. >> >>So I assume the patches for the other MPM's will follow. > > > AFAICT beos and mpmt_os2 don't have any kind of parent/child privilege > separation like the Unix MPMs - so the only one not covered by my patch > is event.
Hm, looks like you are right with this. So no sense in doing something here. Any OS/2 Beos guys around for confirmation? Regards Rüdiger