diff options
| author | Karel Zak | 2014-11-07 10:42:49 +0100 |
|---|---|---|
| committer | Karel Zak | 2014-11-07 10:42:49 +0100 |
| commit | c747564764e612cbdd225701e67f0516fa3c66d4 (patch) | |
| tree | f137f865ba3246e8cb36873abedc23e0add7a1c4 /disk-utils/fdisk-list.c | |
| parent | sfdisk: fix version typos in man page (diff) | |
| download | kernel-qcow2-util-linux-c747564764e612cbdd225701e67f0516fa3c66d4.tar.gz kernel-qcow2-util-linux-c747564764e612cbdd225701e67f0516fa3c66d4.tar.xz kernel-qcow2-util-linux-c747564764e612cbdd225701e67f0516fa3c66d4.zip | |
fdisk: fix and improve -l warnings
Signed-off-by: Karel Zak <kzak@redhat.com>
Diffstat (limited to 'disk-utils/fdisk-list.c')
| -rw-r--r-- | disk-utils/fdisk-list.c | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/disk-utils/fdisk-list.c b/disk-utils/fdisk-list.c index d0d752ef6..824eb48b8 100644 --- a/disk-utils/fdisk-list.c +++ b/disk-utils/fdisk-list.c @@ -89,6 +89,7 @@ void list_disklabel(struct fdisk_context *cxt) const char *bold = NULL; int *ids = NULL; /* IDs of fdisk_fields */ size_t nids = 0, i; + int post = 0; /* print label specific stuff by libfdisk FDISK_ASK_INFO API */ fdisk_list_disklabel(cxt); @@ -170,16 +171,24 @@ void list_disklabel(struct fdisk_context *cxt) } /* print warnings */ + fdisk_reset_iter(itr, FDISK_ITER_FORWARD); while (itr && fdisk_table_next_partition(tb, itr, &pa) == 0) { if (!fdisk_partition_has_start(pa)) continue; - if (!fdisk_lba_is_phy_aligned(cxt, fdisk_partition_get_start(pa))) + if (!fdisk_lba_is_phy_aligned(cxt, fdisk_partition_get_start(pa))) { + if (!post) + fputc('\n', stdout); fdisk_warnx(cxt, _("Partition %zu does not start on physical sector boundary."), fdisk_partition_get_partno(pa) + 1); + post++; + } } - if (fdisk_table_wrong_order(tb)) + if (fdisk_table_wrong_order(tb)) { + if (!post) + fputc('\n', stdout); fdisk_info(cxt, _("Partition table entries are not in disk order.")); + } done: scols_unref_table(out); fdisk_unref_table(tb); |