diff mbox series

[Concept,4/7] efi: Rename var_name64 to name in efi_dump_var_all()

Message ID 20250821153528.141740-5-sjg@u-boot.org
State New
Headers
Series efi: Improvements to env print -e |

Commit Message

Simon Glass Aug. 21, 2025, 3:35 p.m. UTC 
  From: Simon Glass <sjg@chromium.org>

This variable name is quite long. There is only one 'name' (the variable
name) and it has to be u16 as it is an EFI variable. So rename it to
'name'.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

 cmd/nvedit_efi.c | 27 +++++++++++++--------------
 1 file changed, 13 insertions(+), 14 deletions(-)
diff mbox series

Patch

diff --git a/cmd/nvedit_efi.c b/cmd/nvedit_efi.c
index 12c98a5d5c4..0e4abf49fc0 100644
--- a/cmd/nvedit_efi.c
+++ b/cmd/nvedit_efi.c
@@ -146,48 +146,47 @@  out:
 static int efi_dump_var_all(int argc,  char *const argv[],
 			    const efi_guid_t *guid_p, bool verbose, bool nodump)
 {
-	u16 *var_name16, *p;
 	efi_uintn_t buf_size, size;
 	efi_guid_t guid;
 	efi_status_t ret;
 	bool match = false;
+	u16 *name, *p;
 
 	buf_size = 128;
-	var_name16 = malloc(buf_size);
-	if (!var_name16)
+	name = malloc(buf_size);
+	if (!name)
 		return CMD_RET_FAILURE;
 
-	var_name16[0] = 0;
+	name[0] = 0;
 	for (;;) {
 		size = buf_size;
-		ret = efi_get_next_variable_name_int(&size, var_name16,
-						     &guid);
+		ret = efi_get_next_variable_name_int(&size, name, &guid);
 		if (ret == EFI_NOT_FOUND)
 			break;
 		if (ret == EFI_BUFFER_TOO_SMALL) {
 			buf_size = size;
-			p = realloc(var_name16, buf_size);
+			p = realloc(name, buf_size);
 			if (!p) {
-				free(var_name16);
+				free(name);
 				return CMD_RET_FAILURE;
 			}
-			var_name16 = p;
-			ret = efi_get_next_variable_name_int(&size, var_name16,
+			name = p;
+			ret = efi_get_next_variable_name_int(&size, name,
 							     &guid);
 		}
 		if (ret != EFI_SUCCESS) {
-			free(var_name16);
+			free(name);
 			return CMD_RET_FAILURE;
 		}
 
 		if (guid_p && guidcmp(guid_p, &guid))
 			continue;
-		if (!argc || match_name(argc, argv, var_name16)) {
+		if (!argc || match_name(argc, argv, name)) {
 			match = true;
-			efi_dump_single_var(var_name16, &guid, verbose, nodump);
+			efi_dump_single_var(name, &guid, verbose, nodump);
 		}
 	}
-	free(var_name16);
+	free(name);
 
 	if (!match && argc == 1) {
 		printf("Error: \"%s\" not defined\n", argv[0]);