bug-grub
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug #52746] An extra trailing slash is added to ZFS root pool and it's


From: Jozsef Fejes
Subject: [bug #52746] An extra trailing slash is added to ZFS root pool and it's not bootable
Date: Tue, 26 Dec 2017 16:31:25 -0500 (EST)
User-agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/64.0.3282.24 Safari/537.36

Follow-up Comment #1, bug #52746 (project grub):

Some more context:

Grub apparently passes this (invalid) value to initramfs, which is processed
here under Debian:
https://anonscm.debian.org/cgit/pkg-zfsonlinux/zfs.git/tree/contrib/initramfs/scripts/zfs#n770

And this is apparently passed verbatim to zfsonlinux, which dies here:
https://github.com/zfsonlinux/zfs/search?utf8=%E2%9C%93&q=%22trailing+slash+in+name%22&type=

I think it sounds sane that ZOL has this validation. One might argue that the
first link is a bit confusing since their format string includes the slash
unconditionally, so at least their documentation should be fixed. I'm not sure
why this file is under a Debian repo, probably other distros have a different
file with similar functionality?

I'm pretty sure life would be great if update-grub produced a valid name, ie.
without the trailing slash. That should hopefully not break any existing
setup.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?52746>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/




reply via email to

[Prev in Thread] Current Thread [Next in Thread]