Re: [PATCH 35/67] policycoreutils: sandbox: FIXME add level based kill

[Eric Paris <eparis@redhat.com>]

On Fri, 2011-09-16 at 07:33 +0200, Guido Trentalancia wrote:
> On Fri, 2011-09-16 at 06:49 +0200, Guido Trentalancia wrote:
> > On Thu, 2011-09-15 at 15:45 -0400, Daniel J Walsh wrote:
> > > @@ -941,6 +1011,9 @@ childerr:
> > >         waitpid(child, &status, 0);
> > >         status_to_retval(status, status);
> > >  
> > > +       if (execcon && kill_all)
> > > +               killall(execcon);
> > > +
> > >         if (tmpdir_r) cleanup_tmpdir(tmpdir_r, tmpdir_s, pwd, 1);
> > >  
> > >  err: 
> > 
> > Was it not just "kill" instead of "kill_all" ?
> 
> I just realised that variable is going to get renamed in another patch
> ([PATCH 36/67] policycoreutils: sandbox: cntrl-c should kill entire). So
> I think this patch requires 36/67, that's it but was not evident at
> first sight.

Thanks, mistake on my part.  Will move the required change from patch
#36 into patch #35 before committing.

-Eric


--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.