]> err.no Git - util-linux/commitdiff
cfdisk: fix n+1 bug in error message
authorKarel Zak <kzak@redhat.com>
Mon, 9 Aug 2010 09:32:46 +0000 (11:32 +0200)
committerKarel Zak <kzak@redhat.com>
Mon, 9 Aug 2010 09:32:46 +0000 (11:32 +0200)
 # cfdisk -Ps <devname>

correctly uses 1..N range for partition numbers, unfortunately some
error messages use 0..N. This is confusing.

Reported-by: J B <jb.1234abcd@gmail.com>
Signed-off-by: Karel Zak <kzak@redhat.com>
fdisk/cfdisk.c

index e7955fe55946061a75134e1801acd1b29f6910b8..610b84718c2bb82741f7eda8f0079f1938851a9a 100644 (file)
@@ -1629,7 +1629,7 @@ fill_p_info(void) {
                         ((bs <= sectors) ? bs : 0), 1, &errmsg)) {
                    char *bad = _("Bad primary partition");
                    char *msg = (char *) xmalloc(strlen(bad) + strlen(errmsg) + 30);
-                   sprintf(msg, "%s %d: %s", bad, i, errmsg);
+                   sprintf(msg, "%s %d: %s", bad, i + 1, errmsg);
                    fatal(msg, 4);
            }
            if (is_extended(buffer.p.part[i].sys_ind))
@@ -1659,7 +1659,7 @@ fill_p_info(void) {
                             bs, 1, &errmsg)) {
                                char *bad = _("Bad logical partition");
                                char *msg = (char *) xmalloc(strlen(bad) + strlen(errmsg) + 30);
-                               sprintf(msg, "%s %d: %s", bad, i, errmsg);
+                               sprintf(msg, "%s %d: %s", bad, i + 1, errmsg);
                                fatal(msg, 4);
                        }
                }