[PATCH 1/1] cmd: remove superfluous if in eficonfig_edit_boot_option

Ilias Apalodimas ilias.apalodimas at linaro.org
Thu Nov 10 12:44:02 CET 2022


On Thu, 10 Nov 2022 at 13:37, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
>
> Goto for an immediately succeeding label is superfluous.
>
> Fixes: 87d791423ac6 ("eficonfig: menu-driven addition of UEFI boot option")
> Addresses-Coverity: 376202 ("Identical code for different branches")
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> ---
>  cmd/eficonfig.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/cmd/eficonfig.c b/cmd/eficonfig.c
> index 2595dd9563..0dc6e7b01e 100644
> --- a/cmd/eficonfig.c
> +++ b/cmd/eficonfig.c
> @@ -1527,8 +1527,6 @@ static efi_status_t eficonfig_edit_boot_option(u16 *varname, struct eficonfig_bo
>         }
>
>         ret = eficonfig_set_boot_option(varname, final_dp, final_dp_size, bo->description, tmp);
> -       if (ret != EFI_SUCCESS)
> -               goto out;
>  out:
>         free(tmp);
>         free(bo->optional_data);
> --
> 2.37.2
>

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


More information about the U-Boot mailing list