[PATCH 2/4] efi_loader: simplify show_efi_boot_order()

Ilias Apalodimas ilias.apalodimas at linaro.org
Mon May 24 15:52:01 CEST 2021


On Mon, May 24, 2021 at 11:56:54AM +0200, Heinrich Schuchardt wrote:
> To print a UTF-16 string use %ls instead of converting string to UTF-8.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> ---
>  cmd/efidebug.c | 15 +--------------
>  1 file changed, 1 insertion(+), 14 deletions(-)
> 
> diff --git a/cmd/efidebug.c b/cmd/efidebug.c
> index 9444eca49c..4c0facd26c 100644
> --- a/cmd/efidebug.c
> +++ b/cmd/efidebug.c
> @@ -1317,8 +1317,6 @@ static int show_efi_boot_order(void)
>  	u16 var_name16[9], *p16;
>  	void *data;
>  	struct efi_load_option lo;
> -	char *label, *p;
> -	size_t label_len16, label_len;
>  	efi_status_t ret;
> 
>  	size = 0;
> @@ -1380,18 +1378,7 @@ static int show_efi_boot_order(void)
>  			goto out;
>  		}
> 
> -		label_len16 = u16_strlen(lo.label);
> -		label_len = utf16_utf8_strnlen(lo.label, label_len16);
> -		label = malloc(label_len + 1);
> -		if (!label) {
> -			free(data);
> -			ret = CMD_RET_FAILURE;
> -			goto out;
> -		}
> -		p = label;
> -		utf16_utf8_strncpy(&p, lo.label, label_len16);
> -		printf("%2d: %s: %s\n", i + 1, var_name, label);
> -		free(label);
> +		printf("%2d: %s: %ls\n", i + 1, var_name, lo.label);
> 
>  		free(data);
>  	}
> --
> 2.30.2
> 
Reviewed-by: Ilias Apalodimas <ilias.apalodimas at linaro.org>


More information about the U-Boot mailing list