Re: [PATCH] Set size of partition correctly in grub_disk_open()

["Yoshinori K. Okuji" <okuji@enbug.org>]

Hi Jeroen,

On Wednesday 05 July 2006 22:54, Jeroen Dekkers wrote:
>    if (p)
> -    disk->partition = grub_partition_probe (disk, p + 1);
> +    {
> +      disk->partition = grub_partition_probe (disk, p + 1);
> +      if (! disk->partition)
> +	{
> +	  grub_error (GRUB_ERR_UNKNOWN_DEVICE, "no such partition");
> +	  goto fail;
> +	}

I agree with this part.

> +
> +      disk->total_sectors = grub_partition_get_len (disk->partition);
> +    }

But I don't agree with this part. The member "total_sectors" must describe the 
size of the whole disk but not of the partition. This is because GRUB 
accesses partitions through the disk interface. When accessing a partition, 
GRUB tweaks offsets by the start sector of the partition. If not, it is 
another bug.

Thanks,
Okuji