On 1/11/19 9:08 PM, Heinrich Schuchardt wrote:
> On 1/8/19 12:44 AM, Simon Glass wrote:
>> The printf() string here is not actually correct. Add a cast to avoid
>> a warning when checking is enabled.
>>
>> Signed-off-by: Simon Glass <s...@chromium.org>
>> ---
>>
>> Changes in v2: None
>>
>>  lib/efi_loader/efi_file.c | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/lib/efi_loader/efi_file.c b/lib/efi_loader/efi_file.c
>> index 128cb0a627..8a4f3a9f40 100644
>> --- a/lib/efi_loader/efi_file.c
>> +++ b/lib/efi_loader/efi_file.c
>> @@ -221,8 +221,8 @@ static efi_status_t EFIAPI efi_file_open(struct 
>> efi_file_handle *file,
>>      struct file_handle *fh = to_fh(file);
>>      efi_status_t ret;
>>  
>> -    EFI_ENTRY("%p, %p, \"%ls\", %llx, %llu", file, new_handle, file_name,
>> -              open_mode, attributes);
>> +    EFI_ENTRY("%p, %p, \"%ls\", %llx, %llu", file, new_handle,
>> +              (wchar_t *)file_name, open_mode, attributes);
> 
> Our utf-16 strings are all u16[].
> 
> Please, change file_name to be u16* both here and in include/efi_api.h.
> Then check every caller.

Thanks Simon for reporting. I have created a new patch

efi_loader: use u16* for file name
https://lists.denx.de/pipermail/u-boot/2019-January/354375.html

Best regards

Heinrich
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to