[PATCH, natty and maverick] Yama: fix default relationship to check thread group

Brad Figg brad.figg at canonical.com
Fri Apr 8 14:58:53 UTC 2011


On 03/21/2011 02:51 AM, Stefan Bader wrote:
> On 03/18/2011 05:51 PM, Kees Cook wrote:
>> Yet another case of a thread needing to match against the group leader.
>>
>> Signed-off-by: Kees Cook<kees.cook at canonical.com>
>> ---
>>   security/yama/yama_lsm.c |    2 ++
>>   1 files changed, 2 insertions(+), 0 deletions(-)
>>
>> diff --git a/security/yama/yama_lsm.c b/security/yama/yama_lsm.c
>> index b05f1dd..c8f3020 100644
>> --- a/security/yama/yama_lsm.c
>> +++ b/security/yama/yama_lsm.c
>> @@ -173,6 +173,8 @@ static int task_is_descendant(struct task_struct *parent,
>>
>>   	rcu_read_lock();
>>   	read_lock(&tasklist_lock);
>> +	if (!thread_group_leader(parent))
>> +		parent = parent->group_leader;
>>   	while (walker->pid>  0) {
>>   		if (!thread_group_leader(walker))
>>   			walker = walker->group_leader;
>
> Acked-by: Stefan Bader<stefan.bader at canonical.com>
>

The patch looks fine, we need SRU Justification text.

Acked-by: Brad Figg <brad.figg at canonical.com>

-- 
Brad Figg brad.figg at canonical.com http://www.canonical.com




More information about the kernel-team mailing list