libfdisk: sanity check, to prevent overlapping partitions from being partly reported as free
i noticed wrongly reported free space when looking with cfdisk on an USB drive prepared with the latest alpine *.iso[1]. Feel free to apply below patch, which fixed the issue for me. Greetings Fabian [1]: http://dl-cdn.alpinelinux.org/alpine/v3.9/releases/x86/alpine-extended-3.9.0-x86.iso Signed-off-by: Karel Zak <kzak@redhat.com>
This commit is contained in:
parent
a57c9865f8
commit
9a76780f86
|
@ -578,7 +578,7 @@ done:
|
||||||
*
|
*
|
||||||
* Note that free space smaller than grain (see fdisk_get_grain_size()) is
|
* Note that free space smaller than grain (see fdisk_get_grain_size()) is
|
||||||
* ignored.
|
* ignored.
|
||||||
|
*
|
||||||
* Returns: 0 on success, otherwise, a corresponding error.
|
* Returns: 0 on success, otherwise, a corresponding error.
|
||||||
*/
|
*/
|
||||||
int fdisk_get_freespaces(struct fdisk_context *cxt, struct fdisk_table **tb)
|
int fdisk_get_freespaces(struct fdisk_context *cxt, struct fdisk_table **tb)
|
||||||
|
@ -635,7 +635,12 @@ int fdisk_get_freespaces(struct fdisk_context *cxt, struct fdisk_table **tb)
|
||||||
/* add gaps between logical partitions */
|
/* add gaps between logical partitions */
|
||||||
if (fdisk_partition_is_container(pa))
|
if (fdisk_partition_is_container(pa))
|
||||||
rc = check_container_freespace(cxt, parts, *tb, pa);
|
rc = check_container_freespace(cxt, parts, *tb, pa);
|
||||||
last = fdisk_partition_get_end(pa);
|
|
||||||
|
if (fdisk_partition_has_end(pa)) {
|
||||||
|
fdisk_sector_t pa_end = fdisk_partition_get_end(pa);
|
||||||
|
if (pa_end > last)
|
||||||
|
last = fdisk_partition_get_end(pa);
|
||||||
|
}
|
||||||
nparts++;
|
nparts++;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue