[LTP] [PATCH 2/3] commands/keyctl01: Fix getting key serial number

Guangwen Feng fenggw-fnst@cn.fujitsu.com
Thu Jul 13 12:55:33 CEST 2017


Hi!

在 07/11/2017 08:27 PM, Guangwen Feng 写道:
> Hi!
> 
> Thanks for your review.
> 
> 在 07/07/2017 09:08 PM, Cyril Hrubis 写道:
>> Hi!
>>> So fix this by adding a match with "inaccessible".
>>>
>>> Signed-off-by: Guangwen Feng <fenggw-fnst@cn.fujitsu.com>
>>> ---
>>>  testcases/commands/keyctl/keyctl01.sh | 5 +++++
>>>  1 file changed, 5 insertions(+)
>>>
>>> diff --git a/testcases/commands/keyctl/keyctl01.sh b/testcases/commands/keyctl/keyctl01.sh
>>> index 5a97499..8ea2b25 100644
>>> --- a/testcases/commands/keyctl/keyctl01.sh
>>> +++ b/testcases/commands/keyctl/keyctl01.sh
>>> @@ -77,6 +77,11 @@ do_test()
>>>  		fi
>>>  
>>>  		local key=`keyctl show | awk '/debug:fred/ {print $1}'`
>>> +		if [ -z "$key" ]; then
>>> +			key=`keyctl show | \
>>> +				awk -F ':' '/inaccessible/ {print $1}'`
>>> +		fi
>>
>> Can't we rather split the keyctl request and keyctl negate operations
>> into two and get the key after the key has been requested but before it
>> was negated?
>>
>> Or is it required to do the request and negate operation in a signle
>> keyctl command in order to reproduce the kernel crash?
> 
> I think it is required to do the operations in one command...
> I will try to split them and reproduce the kernel crash, thanks.

Without using a single keyctl command, this bug cannot be triggered,
so I want to keep it this way.

I think the problem is that we cannot get the keyid by matching
"debug:fred" via "keyctl show" when the key is expired, I find that
in /proc/keys even the key is expired, we can still see the key's
description, so we can just look it up this way.

I will send a V2 soon.

Best Regards,
Guangwen Feng

> 
> Best Regards,
> Guangwen Feng
> 
>>
>>>  		if [ -n "$key" ]; then
>>>  			keyctl unlink $key @s >/dev/null
>>>  			tst_sleep 50ms
>>> -- 
>>> 1.8.4.2
>>>
>>>
>>>
>>>
>>> -- 
>>> Mailing list info: https://lists.linux.it/listinfo/ltp
>>
> 
> 
> 




More information about the ltp mailing list