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

[Daniel J Walsh <dwalsh@redhat.com>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 09/16/2011 01:33 AM, 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.
> 
> Guido
> 
kill  would conflict with the function kill.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk5zXMsACgkQrlYvE4MpobP5twCdFv8PeXU5xxUK8jVlDh4AEGpC
jjQAnRsPrMSo0umWF4NeSO/+0iKg64TB
=DEO6
-----END PGP SIGNATURE-----

--
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.