Re: [PATCH 1/2] crypto: hisilicon - Fix double free in sec_free_hw_sgl()

2019-09-20 Thread Herbert Xu
On Sun, Sep 15, 2019 at 05:26:56PM +0800, Yunfeng Ye wrote: > There are two problems in sec_free_hw_sgl(): > > First, when sgl_current->next is valid, @hw_sgl will be freed in the > first loop, but it free again after the loop. > > Second, sgl_current and sgl_current->next_sgl is not match when >

[PATCH 1/2] crypto: hisilicon - Fix double free in sec_free_hw_sgl()

2019-09-15 Thread Yunfeng Ye
There are two problems in sec_free_hw_sgl(): First, when sgl_current->next is valid, @hw_sgl will be freed in the first loop, but it free again after the loop. Second, sgl_current and sgl_current->next_sgl is not match when dma_pool_free() is invoked, the third parameter should be the dma address