diff options
author | Heinrich Schuchardt <heinrich.schuchardt@canonical.com> | 2023-02-10 09:19:41 +0100 |
---|---|---|
committer | Heinrich Schuchardt <heinrich.schuchardt@canonical.com> | 2023-02-10 13:05:40 +0100 |
commit | 8d4c2c770c8935d4f1e06f8eb9d6416b8f63ea62 (patch) | |
tree | 3221ba45e7074f1db10268c7c4f4bf5e65b7045a /lib | |
parent | 60e3fedc64532c66f5bcd7c5c3d95e51ab2783e7 (diff) | |
download | u-boot-8d4c2c770c8935d4f1e06f8eb9d6416b8f63ea62.zip u-boot-8d4c2c770c8935d4f1e06f8eb9d6416b8f63ea62.tar.gz u-boot-8d4c2c770c8935d4f1e06f8eb9d6416b8f63ea62.tar.bz2 |
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')
-rw-r--r-- | lib/efi_loader/efi_conformance.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/efi_loader/efi_conformance.c b/lib/efi_loader/efi_conformance.c index 3036d46..0ca26f5 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; |