Jim Meyering <[EMAIL PROTECTED]> writes:
> Otavio Salvador <[EMAIL PROTECTED]> wrote:
>
>> Jim Meyering <[EMAIL PROTECTED]> writes:
>>
>>> static int
>>> msdos_probe (const PedDevice *dev)
>>> {
>>> PedDiskType* disk_type;
>>> - DosRawTable part_table;
>>> + DosRawTable* part_table;
>>> int i;
>>>
>>> PED_ASSERT (dev != NULL, return 0);
>>>
>>> - if (dev->sector_size != 512)
>>> + if (dev->sector_size < sizeof *part_table)
>>> return 0;
>>
>> Shouldn't it be sizeof(DosRawTable)? part_table points to a unkown
>> place at this point or am I missing something?
>
> No. First, "*part_table" as an argument to sizeof doesn't
> cause a dereference. Rather it identifies the type that the
> sizeof operator operates on.
Ah, ok, great! :-)
Ack.
--
O T A V I O S A L V A D O R
---------------------------------------------
E-mail: [EMAIL PROTECTED] UIN: 5906116
GNU/Linux User: 239058 GPG ID: 49A5F855
Home Page: http://otavio.ossystems.com.br
---------------------------------------------
"Microsoft sells you Windows ... Linux gives
you the whole house."
_______________________________________________
parted-devel mailing list
[email protected]
http://lists.alioth.debian.org/mailman/listinfo/parted-devel
