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@xxxxxxxxxxxxx with the words "unsubscribe selinux" without quotes as the message.
