Re: [PATCH v4] kasan/quarantine: fix bugs on qlist_move_cache()

2016-07-05 Thread Joonsoo Kim
On Mon, Jul 04, 2016 at 12:49:08PM +0300, Andrey Ryabinin wrote: > > > On 07/04/2016 07:31 AM, js1...@gmail.com wrote: > > From: Joonsoo Kim > > > > There are two bugs on qlist_move_cache(). One is that qlist's tail > > isn't set properly. curr->next can be NULL since

Re: [PATCH v4] kasan/quarantine: fix bugs on qlist_move_cache()

2016-07-05 Thread Joonsoo Kim
On Mon, Jul 04, 2016 at 12:49:08PM +0300, Andrey Ryabinin wrote: > > > On 07/04/2016 07:31 AM, js1...@gmail.com wrote: > > From: Joonsoo Kim > > > > There are two bugs on qlist_move_cache(). One is that qlist's tail > > isn't set properly. curr->next can be NULL since it is singly linked > >

Re: [PATCH v4] kasan/quarantine: fix bugs on qlist_move_cache()

2016-07-05 Thread Dmitry Vyukov
On Mon, Jul 4, 2016 at 6:31 AM, wrote: > From: Joonsoo Kim > > There are two bugs on qlist_move_cache(). One is that qlist's tail > isn't set properly. curr->next can be NULL since it is singly linked > list and NULL value on tail is invalid if there is

Re: [PATCH v4] kasan/quarantine: fix bugs on qlist_move_cache()

2016-07-05 Thread Dmitry Vyukov
On Mon, Jul 4, 2016 at 6:31 AM, wrote: > From: Joonsoo Kim > > There are two bugs on qlist_move_cache(). One is that qlist's tail > isn't set properly. curr->next can be NULL since it is singly linked > list and NULL value on tail is invalid if there is one item on qlist. > Another one is that

Re: [PATCH v4] kasan/quarantine: fix bugs on qlist_move_cache()

2016-07-04 Thread Andrey Ryabinin
On 07/04/2016 07:31 AM, js1...@gmail.com wrote: > From: Joonsoo Kim > > There are two bugs on qlist_move_cache(). One is that qlist's tail > isn't set properly. curr->next can be NULL since it is singly linked > list and NULL value on tail is invalid if there is one

Re: [PATCH v4] kasan/quarantine: fix bugs on qlist_move_cache()

2016-07-04 Thread Andrey Ryabinin
On 07/04/2016 07:31 AM, js1...@gmail.com wrote: > From: Joonsoo Kim > > There are two bugs on qlist_move_cache(). One is that qlist's tail > isn't set properly. curr->next can be NULL since it is singly linked > list and NULL value on tail is invalid if there is one item on qlist. > Another

[PATCH v4] kasan/quarantine: fix bugs on qlist_move_cache()

2016-07-03 Thread js1304
From: Joonsoo Kim There are two bugs on qlist_move_cache(). One is that qlist's tail isn't set properly. curr->next can be NULL since it is singly linked list and NULL value on tail is invalid if there is one item on qlist. Another one is that if cache is matched,

[PATCH v4] kasan/quarantine: fix bugs on qlist_move_cache()

2016-07-03 Thread js1304
From: Joonsoo Kim There are two bugs on qlist_move_cache(). One is that qlist's tail isn't set properly. curr->next can be NULL since it is singly linked list and NULL value on tail is invalid if there is one item on qlist. Another one is that if cache is matched, qlist_put() is called and it