From: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Date: Sat, 2 Sep 2023 07:35:22 +0000 (+0200)
Subject: cmd: gpt: use UUID accessor more consistently
X-Git-Tag: v2025.01-rc5-pxa1908~847^2~12^2~2
X-Git-Url: http://git.dujemihanovic.xyz/img/static/html/%7B%7B%20%24.Site.BaseURL%20%7D%7Dposts/%7B%7B%20%28.OutputFormats.Get?a=commitdiff_plain;h=396f315520bcbcc2b710ab696326b239577c1ad2;p=u-boot.git

cmd: gpt: use UUID accessor more consistently

disk_partition_uuid() and disk_partition_set_uuid() were introduced to let
us avoid the usage of #ifdef when dealing with the field uuid of
struct disk_partition.

In allocate_disk_part() commit c5f1d005f517 ("part: Add accessors for
struct disk_partition uuid") missed to use the setter.

print_gpt_info() and create_gpt_partitions_list() are further functions
where we should use the getter.

Fixes: c5f1d005f517 ("part: Add accessors for struct disk_partition uuid")
Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
---

diff --git a/cmd/gpt.c b/cmd/gpt.c
index d4068ee63d..96cd8efaaf 100644
--- a/cmd/gpt.c
+++ b/cmd/gpt.c
@@ -220,10 +220,9 @@ static struct disk_part *allocate_disk_part(struct disk_partition *info,
 		UUID_STR_LEN);
 	newpart->gpt_part_info.type_guid[UUID_STR_LEN] = '\0';
 #endif
-	if (IS_ENABLED(CONFIG_PARTITION_UUIDS)) {
-		strlcpy(newpart->gpt_part_info.uuid, disk_partition_uuid(info),
-			UUID_STR_LEN + 1);
-	}
+	if (IS_ENABLED(CONFIG_PARTITION_UUIDS))
+		disk_partition_set_uuid(&newpart->gpt_part_info,
+					disk_partition_uuid(info));
 	newpart->partnum = partnum;
 
 	return newpart;
@@ -262,9 +261,9 @@ static void print_gpt_info(void)
 #ifdef CONFIG_PARTITION_TYPE_GUID
 		printf("Type GUID %s\n", curr->gpt_part_info.type_guid);
 #endif
-#ifdef CONFIG_PARTITION_UUIDS
-		printf("UUID %s\n", curr->gpt_part_info.uuid);
-#endif
+		if (CONFIG_IS_ENABLED(PARTITION_UUIDS))
+			printf("UUID %s\n",
+			       disk_partition_uuid(&curr->gpt_part_info));
 		printf("\n");
 	}
 }
@@ -314,9 +313,12 @@ static int create_gpt_partitions_list(int numparts, const char *guid,
 		strncat(partitions_list, curr->gpt_part_info.type_guid,
 			UUID_STR_LEN + 1);
 #endif
-		strcat(partitions_list, ",uuid=");
-		strncat(partitions_list, curr->gpt_part_info.uuid,
-			UUID_STR_LEN + 1);
+		if (CONFIG_IS_ENABLED(PARTITION_UUIDS)) {
+			strcat(partitions_list, ",uuid=");
+			strncat(partitions_list,
+				disk_partition_uuid(&curr->gpt_part_info),
+				UUID_STR_LEN + 1);
+		}
 		if (curr->gpt_part_info.bootable & PART_BOOTABLE)
 			strcat(partitions_list, ",bootable");
 		strcat(partitions_list, ";");