Hi LaMont,

Would you check this Debian patch and comment if it's okay (or already
included in Git repository) so that this grave bug could be closed.

Thanks,
Jari

    From: Aurelien Jarno <aurel...@aurel32.net>
    http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=510130

    While I agree with the analysis, the patch looks more like a workaround
    than a real solution. The problem is that when free space doesn't start
    on a cylinder boundary (the case for the free space after the volume
    header), there might be less than one cylinder to create a partition.
    The code to ask for partition boundaries is using cylinders (probably
    because partition start should be cylinder aligned) and thus doesn't
    cope with this issue.

    The patch below fixes the issue by making sure free space always starts
    as a cylinder boundary, as anyway that's the only possible location to
    create a new partition.

--- util-linux-2.13.1.1.orig/fdisk/fdisksgilabel.c
+++ util-linux-2.13.1.1/fdisk/fdisksgilabel.c
@@ -501,6 +501,9 @@
                }
                start = sgi_get_start_sector(Index[i])
                        + sgi_get_num_sectors(Index[i]);
+               /* Align free space on cylinder boundary */
+               if (start % cylsize != 0)
+                       start += cylsize - (start % cylsize);
                if (debug > 1) {
                        if (verbose)
                                printf("%2d:%12d\t%12d\t%12d\n", Index[i],



-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to