Hello Bharath and Jean-Luc,
I'm too lazy to write both of you a seperate mail ;)
Both your reports have to do with LVM and I'm not able to repdroduce
them and so to trace them down.
Luckly somebody else now saw a problem in the LVM module code.
Please try if the attached patch helps.
In case you don't know how to do it:
You may need a deb-src line in /etc/apt/sources.list for this apt-get
call:
apt-get build-depends grub2
svn co svn://svn.sv.gnu.org/grub/trunk/grub2
cd grub2
patch -p1 < /path/to/circular-metadata.patch
svn co svn://svn.debian.org/svn/pkg-grub/grub2/trunk/debian
export DH_ALWAYS_EXCLUDE=".svn"
dpkg-buildpackage -b
If you still have problems compiling this then please feel free to send
me a mail in private (i.e. no CC @bugs.debian.org)
Weitergeleitete Nachricht
> Von: Hans Lambermont <[EMAIL PROTECTED]>
>
> Please find the patch attached, this was tested with circular metadata
> and the segfault in grub-setup is gone and the system boots fine.
--
Felix Zielcke
diff -uwr grub-1.96_svn20080813-org/ChangeLog grub-1.96_svn20080813-new/ChangeLog
--- grub-1.96_svn20080813-org/ChangeLog 2008-08-13 17:24:36.0 +0200
+++ grub-1.96_svn20080813-new/ChangeLog 2008-08-29 10:33:03.0 +0200
@@ -1,3 +1,8 @@
+2008-08-28 Hans Lambermont <[EMAIL PROTECTED]> (tiny change)
+ Jan Derk Gerlings <[EMAIL PROTECTED]> (tiny change)
+
+ * disk/lvm.c: Add capability to read circular metadata
+
2008-08-12 Robert Millan <[EMAIL PROTECTED]>
* loader/i386/pc/multiboot.c (grub_multiboot_load_elf32): Move part
diff -uwr grub-1.96_svn20080813-org/disk/lvm.c grub-1.96_svn20080813-new/disk/lvm.c
--- grub-1.96_svn20080813-org/disk/lvm.c 2008-08-28 14:32:53.0 +0200
+++ grub-1.96_svn20080813-new/disk/lvm.c 2008-08-28 18:31:19.0 +0200
@@ -281,7 +281,8 @@
goto fail;
}
- metadatabuf = grub_malloc (mda_size);
+ /* alloc for circular worst-case scenario */
+ metadatabuf = grub_malloc (2*mda_size);
if (! metadatabuf)
goto fail;
@@ -300,6 +301,12 @@
}
rlocn = mdah->raw_locns;
+ if (rlocn->offset + rlocn->size > mdah->size)
+{
+ /* metadata is circular */
+ grub_memcpy(metadatabuf + mda_size, metadatabuf + mdah->start,
+ ((rlocn->offset + rlocn->size) - mdah->size));
+}
p = q = metadatabuf + grub_le_to_cpu64 (rlocn->offset);
while (*q != ' ' && q < metadatabuf + mda_size)