From 1a30ddeda9a5120dca0a9b9c9673d24b375cdc34 Mon Sep 17 00:00:00 2001 From: Heinrich Schuchardt Date: Tue, 4 Nov 2025 23:27:12 +0100 Subject: efi_client: efi_store_memory_map() must return int The type efi_status_t is not compatible with the return type int. Let efi_store_memory_map() return -EFAULT instead of a truncated EFI error code. Acked-by: Ilias Apalodimas Signed-off-by: Heinrich Schuchardt --- lib/efi_client/efi.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'lib') diff --git a/lib/efi_client/efi.c b/lib/efi_client/efi.c index bcb34d67465..a3a40b5549d 100644 --- a/lib/efi_client/efi.c +++ b/lib/efi_client/efi.c @@ -155,7 +155,7 @@ int efi_store_memory_map(struct efi_priv *priv) putc(' '); printhex2(ret); puts(" No memory map\n"); - return ret; + return -EFAULT; } /* * Since doing a malloc() may change the memory map and also we want to @@ -168,7 +168,7 @@ int efi_store_memory_map(struct efi_priv *priv) if (!priv->memmap_desc) { printhex2(ret); puts(" No memory for memory descriptor\n"); - return ret; + return -EFAULT; } ret = boot->get_memory_map(&priv->memmap_size, priv->memmap_desc, @@ -177,7 +177,7 @@ int efi_store_memory_map(struct efi_priv *priv) if (ret) { printhex2(ret); puts(" Can't get memory map\n"); - return ret; + return -EFAULT; } return 0; -- cgit v1.2.3 From 35e510f2af576de9abd17660da759d25a9c45e0a Mon Sep 17 00:00:00 2001 From: Heinrich Schuchardt Date: Wed, 5 Nov 2025 00:29:33 +0100 Subject: efi_driver: correct formatting in efi_uc_stop() Correct indentation. Reviewed-by: Bin Meng Signed-off-by: Heinrich Schuchardt --- lib/efi_driver/efi_uclass.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/efi_driver/efi_uclass.c b/lib/efi_driver/efi_uclass.c index 7392c60f0f9..bd46c9f5539 100644 --- a/lib/efi_driver/efi_uclass.c +++ b/lib/efi_driver/efi_uclass.c @@ -227,7 +227,7 @@ static efi_status_t EFIAPI efi_uc_stop( goto out; } ret = EFI_SUCCESS; - goto out; + goto out; } /* Destroy all children */ -- cgit v1.2.3 From 59f2c5a0fa65598c911306a7873638ef516b6e6f Mon Sep 17 00:00:00 2001 From: Heinrich Schuchardt Date: Wed, 5 Nov 2025 02:30:45 +0100 Subject: efi_driver: typo 'to be write' %s/to be write/to write/ Reviewed-by: Bin Meng Signed-off-by: Heinrich Schuchardt --- lib/efi_driver/efi_block_device.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/efi_driver/efi_block_device.c b/lib/efi_driver/efi_block_device.c index 070747de515..913755336b2 100644 --- a/lib/efi_driver/efi_block_device.c +++ b/lib/efi_driver/efi_block_device.c @@ -84,7 +84,7 @@ static ulong efi_bl_read(struct udevice *dev, lbaint_t blknr, lbaint_t blkcnt, * efi_bl_write() - write to block device * * @dev: device - * @blknr: first block to be write + * @blknr: first block to write * @blkcnt: number of blocks to write * @buffer: input buffer * Return: number of blocks transferred -- cgit v1.2.3 From dbed053487610781bdf48ca687fef9e39a982cb4 Mon Sep 17 00:00:00 2001 From: Heinrich Schuchardt Date: Wed, 5 Nov 2025 13:24:26 +0100 Subject: efi_driver: don't leak name in efi_bl_create_block_device() blk_create_devicef() uses a copy of parameter name. We can use a local variable. Reviewed-by: Bin Meng Reviewed-by: Ilias Apalodimas Signed-off-by: Heinrich Schuchardt --- lib/efi_driver/efi_block_device.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) (limited to 'lib') diff --git a/lib/efi_driver/efi_block_device.c b/lib/efi_driver/efi_block_device.c index 913755336b2..9ab956c7ef7 100644 --- a/lib/efi_driver/efi_block_device.c +++ b/lib/efi_driver/efi_block_device.c @@ -124,7 +124,7 @@ efi_bl_create_block_device(efi_handle_t handle, void *interface) efi_status_t ret; int r; int devnum; - char *name; + char name[18]; /* strlen("efiblk#2147483648") + 1 */ struct efi_block_io *io = interface; struct efi_blk_plat *plat; @@ -136,9 +136,6 @@ efi_bl_create_block_device(efi_handle_t handle, void *interface) if (devnum < 0) return EFI_OUT_OF_RESOURCES; - name = calloc(1, 18); /* strlen("efiblk#2147483648") + 1 */ - if (!name) - return EFI_OUT_OF_RESOURCES; sprintf(name, "efiblk#%d", devnum); /* Create driver model udevice for the EFI block io device */ @@ -146,7 +143,6 @@ efi_bl_create_block_device(efi_handle_t handle, void *interface) devnum, io->media->block_size, (lbaint_t)io->media->last_block, &bdev)) { ret = EFI_OUT_OF_RESOURCES; - free(name); goto err; } -- cgit v1.2.3 From 6bb374b1a1226b964db8476b2939a280c7477e4a Mon Sep 17 00:00:00 2001 From: Heinrich Schuchardt Date: Tue, 4 Nov 2025 22:48:04 +0100 Subject: efi_loader: typo 'mange' in efi_net.c %s/mange/manage/ Reviewed-by: Ilias Apalodimas Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_net.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/efi_loader/efi_net.c b/lib/efi_loader/efi_net.c index b8a6e08ba8e..0f8a851e3f2 100644 --- a/lib/efi_loader/efi_net.c +++ b/lib/efi_loader/efi_net.c @@ -372,7 +372,7 @@ out: } /* - * efi_net_receive_filters() - mange multicast receive filters + * efi_net_receive_filters() - manage multicast receive filters * * This function implements the ReceiveFilters service of the * EFI_SIMPLE_NETWORK_PROTOCOL. See the Unified Extensible Firmware Interface -- cgit v1.2.3