Joel Granados <[email protected]> wrote:

> I revisited this issue and conclude that I was very hasty in posting the 
> patch.  Xerces is correct,  the creation of the msdos partition should be 
> allowed.  Its possible, so why not?.  What should not be allowed is the 
> creation of a partition inside the msdos label that has length or start 
> values that exceed 2T.  Why,  because the values used for these partitions 
> would not be the ones given by the user.  This is how parted does things ATM, 
> so everything is ok.
>
> I still, however, see that the code that checks if the partition has exceeded 
> its limits, is in libparted/disk.c.  I still insist that it should be 
> contained in every label file.  Or at least have some sort of variable in 
> each lable that defines the maximum allowable partition size and partition 
> start.  In this way, these max values can be called from libparted/disk.c and 
> used in a more general way.  Right know we are defining the maximum size and 
> start of an msdos partition in disk.c,  that just seems wrong to me.
>
> Comments greatly appreciated.  otavio. meyering?

Good points.
I agree.

_______________________________________________
parted-devel mailing list
[email protected]
http://lists.alioth.debian.org/mailman/listinfo/parted-devel

Reply via email to