[PATCH 1/1] efi_loader: memory leak efi_add_memory_map_pg

[Ilias Apalodimas]

On Sun, 30 Jul 2023 at 14:05, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
>
> Don't leak newlist if we error out.
>
> Fixes: 74c16acce30b ("efi_loader: Don't allocate from memory holes")
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> ---
>  lib/efi_loader/efi_memory.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/lib/efi_loader/efi_memory.c b/lib/efi_loader/efi_memory.c
> index 45d3bf52b8..ebf4a2d5fa 100644
> --- a/lib/efi_loader/efi_memory.c
> +++ b/lib/efi_loader/efi_memory.c
> @@ -324,6 +324,7 @@ static efi_status_t efi_add_memory_map_pg(u64 start, u64 pages,
>                                  * The user requested to only have RAM overlaps,
>                                  * but we hit a non-RAM region. Error out.
>                                  */
> +                               free(newlist);
>                                 return EFI_NO_MAPPING;
>                         case EFI_CARVE_NO_OVERLAP:
>                                 /* Just ignore this list entry */
> @@ -354,6 +355,7 @@ static efi_status_t efi_add_memory_map_pg(u64 start, u64 pages,
>                  * The payload wanted to have RAM overlaps, but we overlapped
>                  * with an unallocated region. Error out.
>                  */
> +               free(newlist);
>                 return EFI_NO_MAPPING;
>         }
>
> --
> 2.40.1
>

Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>