On Fri, Nov 08, 2024 at 06:22:27PM +0100, Christophe JAILLET wrote: > If do_cpt_init() fails, a previous dma_alloc_coherent() call needs to be > undone. > > Add the needed dma_free_coherent() before returning. > > Fixes: 9e2c7d99941d ("crypto: cavium - Add Support for Octeon-tx CPT Engine") > Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> > --- > Compile tested only. > > I guess that dmam_alloc_coherent() could also be used to fix the leak. > This would simplify cpt_unload_microcode(), IIUC. > --- > drivers/crypto/cavium/cpt/cptpf_main.c | 2 ++ > 1 file changed, 2 insertions(+) Patch applied. Thanks. -- Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt