Re: apparmor: double lock

From: John Johansen
Date: Mon Sep 06 2010 - 13:13:24 EST


On 09/05/2010 05:29 PM, James Morris wrote:
> On Sat, 4 Sep 2010, John Johansen wrote:
>
>> On 09/04/2010 04:57 AM, Jiri Slaby wrote:
>>> Hi,
>>>
>>> stanse found that you possibly double lock ns->lock:
>>> aa_remove_profiles
>>> -> write_lock(&ns->lock)
>>> -> __remove_namespace(ns)
>>> -> destroy_namespace(ns)
>>> -> write_lock(&ns->lock)
>>>
>>> Could you fix that if that may happen at all?
>>>
>> Yes, thanks Jiri, the patch has already been posted
>>
>> http://marc.info/?l=linux-kernel&m=128295924407596&w=2
>
> Are all of the patches in that set ready to be merged to Linus?
>
yes they should be now, I believe so.

I have just posted out the updated Fix security_task_setrlimit logic
patch that fixes the commenting. I can repost the series if you would
like.

thanks
john
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/