Re: [PATCH] crypto: dh - fix memory leak

2018-07-19 Thread Herbert Xu
On Tue, Jul 10, 2018 at 09:22:52AM -0500, Gustavo A. R. Silva wrote: > In case memory resources for *base* were allocated, release them > before return. > > Addresses-Coverity-ID: 1471702 ("Resource leak") > Fixes: e3fe0ae12962 ("crypto: dh - add public key verification test") > Signed-off-by: Gus

Re: [PATCH] crypto: dh - fix memory leak

2018-07-10 Thread Gustavo A. R. Silva
Hi Stephan, On 07/10/2018 10:28 AM, Stephan Müller wrote: > Am Dienstag, 10. Juli 2018, 16:22:52 CEST schrieb Gustavo A. R. Silva: > > Hi Gustavo, > >> In case memory resources for *base* were allocated, release them >> before return. > > Thanks a lot. > Glad to help. :) > Reviewed-by: Steph

Re: [PATCH] crypto: dh - fix memory leak

2018-07-10 Thread Stephan Müller
Am Dienstag, 10. Juli 2018, 16:22:52 CEST schrieb Gustavo A. R. Silva: Hi Gustavo, > In case memory resources for *base* were allocated, release them > before return. Thanks a lot. Reviewed-by: Stephan Müller Ciao Stephan

[PATCH] crypto: dh - fix memory leak

2018-07-10 Thread Gustavo A. R. Silva
In case memory resources for *base* were allocated, release them before return. Addresses-Coverity-ID: 1471702 ("Resource leak") Fixes: e3fe0ae12962 ("crypto: dh - add public key verification test") Signed-off-by: Gustavo A. R. Silva --- crypto/dh.c | 2 +- 1 file changed, 1 insertion(+), 1 dele