]> git.ipfire.org Git - thirdparty/util-linux.git/commit
libfdisk: Distinguish between first LBA sector and partition
authorTobias Stoeckmann <tobias@stoeckmann.org>
Sat, 27 Aug 2016 17:28:48 +0000 (19:28 +0200)
committerKarel Zak <kzak@redhat.com>
Mon, 29 Aug 2016 11:22:24 +0000 (13:22 +0200)
commit1037269fec43e8055c530c13d4fd58819e86b233
treea2c35d1f38fc75b4a16badfb859fd3f80da44f95
parent848018183869d908ddfa59ce7362272f65baa860
libfdisk: Distinguish between first LBA sector and partition

An insufficient check leads to an invalid free space output, as seen here:

$ dd if=/dev/zero of=cfdisk.iso bs=1M count=1
$ losetup -f cfdisk.iso
$ echo w | fdisk /dev/loop0
$ echo '1,1' | sfdisk /dev/loop0 --append
$ echo '3,' | sfdisk /dev/loop0 --append
$ sfdisk --list-free /dev/loop0
Start End Sectors Size
    1   2       2   1K
$ _

In this case, libfdisk fails to notice that it tries to calculate space
between two partitions, not between start of disk and first partition.
Currently, the code tries to achieve that by checking the address of the
last "partition", which is the first_lba block.  Now if the first
partition is merely 1 block in size, the "last" address is still equal
to the first_lba block, which renders the check in libfdisk for the next
partition invalid.

I chose to use "nparts == 0" for this check, because the partitions are
properly sorted before iterating over them.

Signed-off-by: Tobias Stoeckmann <tobias@stoeckmann.org>
libfdisk/src/table.c