[U-Boot] [PATCH 6/8] rsa: Fix missing memory leak on error in fdt_add_bignum()
Heinrich Schuchardt
xypron.glpk at gmx.de
Sat Jun 9 19:50:42 UTC 2018
On 06/09/2018 08:22 PM, Simon Glass wrote:
> Thsi function can fail without freeing all its memory. Fix it.
>
> Reported-by: Coverity (CID: 131217)
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> lib/rsa/rsa-sign.c | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/lib/rsa/rsa-sign.c b/lib/rsa/rsa-sign.c
> index d2788bf79a..2a09d2b19e 100644
> --- a/lib/rsa/rsa-sign.c
> +++ b/lib/rsa/rsa-sign.c
> @@ -667,15 +667,13 @@ static int fdt_add_bignum(void *blob, int noffset, const char *prop_name,
> * might fail several times
Please, fix the memory leaks above this line too.
Best regards
Heinrich
> */
> ret = fdt_setprop(blob, noffset, prop_name, buf, size);
> - if (ret)
> - return -FDT_ERR_NOSPACE;
> free(buf);
> BN_free(tmp);
> BN_free(big2);
> BN_free(big32);
> BN_free(big2_32);
>
> - return ret;
> + return ret ? -FDT_ERR_NOSPACE : 0;
> }
>
> int rsa_add_verify_data(struct image_sign_info *info, void *keydest)
>
More information about the U-Boot
mailing list