On Fri, Sep 20, 2019 at 4:59 PM Eugeniu Rosca erosca@de.adit-jv.com wrote:
From: Roman Stratiienko roman.stratiienko@globallogic.com
At this point we are using part number sub-command to retrieve UUID of the partition using it's name.
e.g.: part number mmc $mmcdev system_a system_a_index part uuid mmc $mmcdev:${system_a_index} system_a_uuid
Since 'part uuid' sub-command expects partition index in hex format and 'part number' returns decimal value, 'part uuid' command will provide wrong UUID or fail.
Fixes: be683756f62034 ("cmd: part: Add 'number' sub-command") Cc: Dirk Behme dirk.behme@de.bosch.com Reported-by: Pontus Fuchs pontus.fuchs@se.bosch.com Signed-off-by: Roman Stratiienko roman.stratiienko@globallogic.com Signed-off-by: Eugeniu Rosca erosca@de.adit-jv.com
cmd/part.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/cmd/part.c b/cmd/part.c index 6cfb67b27959..5e4e45ca6d8b 100644 --- a/cmd/part.c +++ b/cmd/part.c @@ -152,7 +152,7 @@ static int do_part_info(int argc, char * const argv[], enum cmd_part_info param) snprintf(buf, sizeof(buf), LBAF, info.size); break; case CMD_PART_INFO_NUMBER:
snprintf(buf, sizeof(buf), "%d", part);
snprintf(buf, sizeof(buf), "0x%x", part); break; default: printf("** Unknown cmd_part_info value: %d\n", param);-- 2.23.0
Reviewed-by: Igor Opaniuk igor.opaniuk@gmail.com