On Fri, Sep 04, 2009 at 08:52:44PM +0200, Felix Zielcke wrote:
> r2558 actually enabled this code path which was (almost) never executed
> before in probe():
>
> if (S_ISREG (st.st_mode))
> {
> /* Regular file. Verify that we can read it properly. */
>
> grub_file_t file;
> grub_util_info ("reading %s via OS facilities", path);
> filebuf_via_sys = grub_util_read_image (path);
>
> grub_util_info ("reading %s via GRUB facilities", path);
> asprintf (&grub_path, "(%s)%s", drive_name, path);
> file = grub_file_open (grub_path);
> filebuf_via_grub = xmalloc (file->size);
> grub_file_read (file, filebuf_via_grub, file->size);
>
> grub_util_info ("comparing");
>
> if (memcmp (filebuf_via_grub, filebuf_via_sys, file->size))
> grub_util_error ("files differ");
> }
> printf ("%s\n", fs->name);
>
> If /boot is on a seperate partition and you run `grub-probe -t
> fs /boot/grub/core.img' grub_file_open returns 0 because grub_path is
> the full system path and not a relative path which GRUB needs.
A proper fix for this would be too intrusive for 1.97. Unless someone
has a bright idea, I'll comment it out untill we've released.
--
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
