[apparmor] [PATCH 04/27] apparmor: fix error code to failure message mapping for name lookup

John Johansen john.johansen at canonical.com
Wed Nov 21 17:27:16 UTC 2012


On 11/21/2012 06:54 AM, Steve Beattie wrote:
> On Tue, Nov 20, 2012 at 08:39:44PM -0800, John Johansen wrote:
>> -ESTALE used to be incorrectly used to indicate a disconnected path, when
>> name lookup failed.  This was fixed in commit e1b0e444 to correctly return
>> -EACCESS, but the error to failure message mapping was not correctly updated
>> to reflect this change.
>>
>> Signed-off-by: John Johansen <john.johansen at canonical.com>
> Acked-by: Steve Beattie <sbeattie at ubuntu.com>
> 
> As an aside, I remembered asking a question about needing to keep
> the reverse patch around for older kernels that returned -ESTALE, and
> found it in the archives. Turned out I had already acked this patch
> and patches 1 and 2 the prior time you sent them to the list. Any
> chance you can hang on to the acked-by statements or is that not
> possible in git?
> 
I can sorry, I'll make sure to go through and add the acks this time

>> ---
>>  security/apparmor/path.c |    2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/security/apparmor/path.c b/security/apparmor/path.c
>> index e91ffee..35b394a 100644
>> --- a/security/apparmor/path.c
>> +++ b/security/apparmor/path.c
>> @@ -174,7 +174,7 @@ static int get_name_to_buffer(struct path *path, int flags, char *buffer,
>>  	if (info && error) {
>>  		if (error == -ENOENT)
>>  			*info = "Failed name lookup - deleted entry";
>> -		else if (error == -ESTALE)
>> +		else if (error == -EACCES)
>>  			*info = "Failed name lookup - disconnected path";
>>  		else if (error == -ENAMETOOLONG)
>>  			*info = "Failed name lookup - name too long";
> 
> 
> 




More information about the AppArmor mailing list