Re: [PATCH v3] crypto: ixp4xx - Fix error handling path in 'aead_perform()'

2017-08-02 Thread Arnd Bergmann
On Wed, Aug 2, 2017 at 10:40 AM, Herbert Xu wrote: > On Thu, Jul 20, 2017 at 09:37:10AM +0200, Arnd Bergmann wrote: > > Oops, looks like I introduced the bug. Anyway, such is the danger > of fixing compiler warnings in rarely used code. > > For some reason your patch is corrupted in patchwork. A

[PATCH v3] crypto: ixp4xx - Fix error handling path in 'aead_perform()'

2017-08-02 Thread Herbert Xu
On Thu, Jul 20, 2017 at 09:37:10AM +0200, Arnd Bergmann wrote: > > Thanks for spotting my mistake! > > I've looked at it again and think it's unfortunately still wrong with > your patch, > as there is a 'goto free_buf_src' after dma_pool_alloc(), and that now needs > to jump to free_buf_dst instea