At Wed, 6 Feb 2008 17:45:07 +0100,
Robert Millan wrote:
> Unless I missed something, it seems that grub_raid_array contains redundant
> information (`name' is already present via `disk->name'). I propose to
> simplify it this way.
No idea why, I don't have the time to look at the actual code, but
> @@ -410,7 +410,7 @@ grub_raid_scan_device (const char *name)
> return 0;
> }
>
> - if (array->device[sb.this_disk.number].name != 0)
> + if (array->device[sb.this_disk.number]->name != 0)
> {
> /* We found multiple devices with the same number. Again,
> this shouldn't happen.*/
looks suspicious to me. Is that really doing what it is meant to do?
Regards,
Jeroen Dekkers
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel
