[apparmor] [PATCH 3/5] Make setprocattr more abstract so it will work for any pid

John Johansen john.johansen at canonical.com
Tue Jul 12 19:06:34 UTC 2011


On 07/12/2011 12:01 PM, Kees Cook wrote:
> Hi John,
> 
> On Tue, Jul 12, 2011 at 11:49:03AM -0700, John Johansen wrote:
>> -	rc = setprocattr("/proc/%d/attr/exec", buf, len);
>> +	rc = setprocattr(aa_gettid(), "/proc/%d/attr/exec", buf, len);
>> [...]
>> -	rc = setprocattr("/proc/%d/attr/current", buf, pos - buf);
>> +	rc = setprocattr(aa_gettid(), "/proc/%d/attr/current", buf, pos - buf);
> 
> Shouldn't these be "exec" and "current", not the whole path?
> 
yep, thanks not sure I missed that




More information about the AppArmor mailing list