[PATCH 1/3] cmd: fix gpt setenv

Heinrich Schuchardt heinrich.schuchardt at canonical.com
Sat Aug 26 03:53:41 CEST 2023


Do not assume that partitions are continuously numbered starting at 1.

Having a partition table with a single partition 63 is valid.

Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
---
 cmd/gpt.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/cmd/gpt.c b/cmd/gpt.c
index 007a68eaa7..d0e165d539 100644
--- a/cmd/gpt.c
+++ b/cmd/gpt.c
@@ -786,10 +786,8 @@ static int gpt_setenv(struct blk_desc *desc, const char *name)
 
 		for (i = 1; i < part_drv->max_entries; i++) {
 			ret = part_drv->get_info(desc, i, &pinfo);
-			if (ret) {
-				/* no more entries in table */
-				break;
-			}
+			if (ret)
+				continue;
 
 			if (!strcmp(name, (const char *)pinfo.name)) {
 				/* match found, setup environment variables */
-- 
2.40.1



More information about the U-Boot mailing list