On Fri, Jul 17, 2009 at 06:41:59PM +0200, Vladimir 'phcoder' Serbinenko wrote:
> Sometimes a media that can be partioned isn't really partioned. E.g.
> usb sticks. This is a patch to handle this situation.

But we had a check for this already, is it not working?

  if (! dest_dev->disk->has_partitions)
    { 
      grub_util_warn ("Attempting to install GRUB to a partitionless disk.  
This is a BAD idea.");
      goto unable_to_embed;
    }

> Unfortunately
> such medium is often formated with a flavour of FAT which shares its
> signature with MBR so it may be easily misidentified as
> pc_partition_table. Furthermore the same signature is shared with
> bootsectors including grub. One possibility is to try interpret disk
> as known filesystems and see if we succeed. But the problem is that
> the check for FAT are light and may result in false positives too. The
> only more or less advanced check there is a check for FATXX string.
> But I was about to propose to eliminate this check since I encountered
> a FAT filesystem without this string on friend's SD card formatted
> with symbian which he wanted to use as liveusb. Does anyone has a
> better idea?

I'm not sure there's much we can do about this.  Using heuristics sounds like
it will make the solution worse than the problem.  I don't care much about
Microsoft filesystems, but I'd hate to see GRUB fail on a completely sane
ext3 inside msdos label because it happened to look like FAT in raw disk at
the same time.

-- 
Robert Millan

  The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and
  how) you may access your data; but nobody's threatening your freedom: we
  still allow you to remove your data and not access it at all."


_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to