[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v1 1/1] riscv/boot: Fix possible memory leak
From: |
Bin Meng |
Subject: |
Re: [PATCH v1 1/1] riscv/boot: Fix possible memory leak |
Date: |
Thu, 3 Oct 2019 08:52:58 +0800 |
On Thu, Oct 3, 2019 at 5:38 AM Alistair Francis
<address@hidden> wrote:
>
> Coverity (CID 1405786) thinks that there is a possible memory leak as
> we don't guarentee that the memory allocatd from riscv_find_firmware()
> is freed. This is a false positive, but let's tidy up the code to fix
> the warning.
>
> Signed-off-by: Alistair Francis <address@hidden>
> ---
> hw/riscv/boot.c | 11 ++++-------
> 1 file changed, 4 insertions(+), 7 deletions(-)
>
Reviewed-by: Bin Meng <address@hidden>
Thanks for the patch. I am not sure how I can easily run Coverity to
verify the fix though.
Regards,
Bin