Thread (8 messages) 8 messages, 2 authors, 2021-09-23

Re: [PATCH v2] crypto: sun8i-ce: fix multiple memory leaks in sun8i_ce_hash_run

From: Herbert Xu <herbert@gondor.apana.org.au>
Date: 2021-08-12 10:05:37
Also in: linux-arm-kernel, linux-sunxi, lkml

On Tue, Aug 03, 2021 at 02:31:38PM +0800, Dongliang Mu wrote:
-theend:
-	kfree(buf);
+err_result:
 	kfree(result);
-	crypto_finalize_hash_request(engine, breq, err);
+err_buf:
+	kfree(buf);
+out:
+	if (!err)
+		crypto_finalize_hash_request(engine, breq, err);
 	return 0;
This does not look right.  You're returning zero in case of an error

Cheers,
-- 
Email: Herbert Xu [off-list ref]
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help