summaryrefslogtreecommitdiff
path: root/lib/efi_loader/efi_conformance.c
diff options
context:
space:
mode:
authorHeinrich Schuchardt <heinrich.schuchardt@canonical.com>2023-02-10 09:19:41 +0100
committerHeinrich Schuchardt <heinrich.schuchardt@canonical.com>2023-02-10 13:05:40 +0100
commit8d4c2c770c8935d4f1e06f8eb9d6416b8f63ea62 (patch)
tree3221ba45e7074f1db10268c7c4f4bf5e65b7045a /lib/efi_loader/efi_conformance.c
parent60e3fedc64532c66f5bcd7c5c3d95e51ab2783e7 (diff)
downloadu-boot-8d4c2c770c8935d4f1e06f8eb9d6416b8f63ea62.tar.gz
efi_loader: fix efi_ecpt_register()
num_entries should be unsigned to avoid warnings. As the target field is u16 we should use this type. lib/efi_loader/efi_conformance.c: In function ‘efi_ecpt_register’: lib/efi_loader/efi_conformance.c:30:33: warning: conversion to ‘long unsigned int’ from ‘int’ may change the sign of the result [-Wsign-conversion] 30 | ecpt_size = num_entries * sizeof(efi_guid_t) | ^ lib/efi_loader/efi_conformance.c:46:36: warning: conversion from ‘int’ to ‘u16’ {aka ‘short unsigned int’} may change value [-Wconversion] 46 | ecpt->number_of_profiles = num_entries; | ^~~~~~~~~~~ Fixes: 6b92c1735205 ("efi: Create ECPT table") Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Diffstat (limited to 'lib/efi_loader/efi_conformance.c')
-rw-r--r--lib/efi_loader/efi_conformance.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/efi_loader/efi_conformance.c b/lib/efi_loader/efi_conformance.c
index 3036d46349..0ca26f57a7 100644
--- a/lib/efi_loader/efi_conformance.c
+++ b/lib/efi_loader/efi_conformance.c
@@ -22,7 +22,7 @@ static const efi_guid_t efi_ebbr_2_1_guid =
*/
efi_status_t efi_ecpt_register(void)
{
- int num_entries = 0;
+ u16 num_entries = 0;
struct efi_conformance_profiles_table *ecpt;
efi_status_t ret;
size_t ecpt_size;