Dear Alison,

In message <1497137617-772-1-git-send-email-ali...@peloton-tech.com> you wrote:
> 
> This patch provides support in u-boot for renaming GPT
> partitions.  The renaming is accomplished via new 'gpt swap'
> and 'gpt rename' commands.

Thanks.

One question: can multiple GPT partitions have the same name?

> The 'swap' mode prints a warning if no matching partition names
> are found.  If only one matching name of a provided pair is found, it
> renames the matching partitions to the new name.

I see a problem here...

> +     if (!strcmp(subcomm, "swap")) {
> +             if ((strlen(name1) > PART_NAME_LEN) || (strlen(name2) > 
> PART_NAME_LEN)) {
> +                     printf("Names longer than %d characters are 
> truncated.\n", PART_NAME_LEN);
> +                     return -EINVAL;
> +             }
> +             list_for_each(pos, &disk_partitions) {
> +                     curr = list_entry(pos, struct disk_part, list);
> +                     if (!strcmp((char *)curr->gpt_part_info.name, name1)) {
> +                             strcpy((char *)curr->gpt_part_info.name, name2);
> +                             changed++;
> +                     }
> +                     else if (!strcmp((char *)curr->gpt_part_info.name, 
> name2)) {
> +                             strcpy((char *)curr->gpt_part_info.name, name1);
> +                             changed++;
> +                     }
> +
> +             }
> +             if (changed == 0) {
> +                     printf("No matching partition names were found.\n");
> +                     return ret;
> +             }

You will never know if there really was a pair of names that was
swapped.  Just a single rename of name1->name2 _or_ of name2->name1
will make the user think everything was fine.

[Note: I'm in the process of relocating and will be offline for the
next 2..3. days.  Don't expect more comments from mee soon.
Sorry...]

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
"I dislike companies that have a we-are-the-high-priests-of-hardware-
so-you'll-like-what-we-give-you attitude. I like commodity markets in
which iron-and-silicon hawkers know that they exist to  provide  fast
toys for software types like me to play with..."    - Eric S. Raymond
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to