Re: [PATCH] kasan/quarantine: fix NULL pointer dereference bug

From: Joonsoo Kim
Date: Fri Jul 01 2016 - 09:58:01 EST


2016-07-01 17:11 GMT+09:00 Andrey Ryabinin <aryabinin@xxxxxxxxxxxxx>:
>
>
> On 07/01/2016 10:53 AM, js1304@xxxxxxxxx wrote:
>> From: Joonsoo Kim <iamjoonsoo.kim@xxxxxxx>
>>
>> If we move an item on qlist's tail, we need to update qlist's tail
>> properly. curr->next can be NULL since it is singly linked list
>> so it is invalid for tail. curr is scheduled to be moved so
>> using prev would be correct.
>
> Hmm.. prev may be the element that moved in 'to' list. We need to assign the last element
> from which is in ther 'from' list.

You're right. Also, I find another bug on this function.
I manage them on v2 and sent.

Thanks.