bugGNU GRUB - Bugs: bug #47681, 30_os-prober doesn't handle...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #47681: 30_os-prober doesn't handle multiple initrd paths

Submitter:  General Chaos <general_chaos>
Submitted:  Tue 12 Apr 2016 10:28:52 PM UTC
Votes: 1
 
Category:  Configuration Severity:  Major
Priority:  5 - Normal Item Group:  None
Status:  None Privacy:  Public
Assigned to:  None Originator Name: 
Open/Closed:  Open Release:  Git master
Release:  Reproducibility:  Every Time
Planned Release:  None

Sat 25 Sep 2021 11:11:02 PM UTC, comment #3: 

comment #2:

> Please update this bug report when os-prober is fixed. Suggested solution is OK.


It appears that the issue on os-prober's end is fixed with https://salsa.debian.org/installer-team/os-prober/-/commit/540e9b5b3c6f29a9f342c7b618781e87343b5728

Peter Levine <nsane457>
Wed 13 Apr 2016 06:02:48 AM UTC, comment #2: 

Please update this bug report when os-prober is fixed. Suggested solution is OK.

Andrei Borzenkov <arvidjaar>
Group Member
Tue 12 Apr 2016 11:23:09 PM UTC, comment #1: 

Debian bug report for linux-boot-prober (in package os-prober): https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=820838

General Chaos <general_chaos>
Tue 12 Apr 2016 10:28:52 PM UTC, original submission:  

Some distros, e.g. Manjaro, use multiple paths on the initrd line.
Including only the first results in a kernel panic rather than booting.

30_os-prober fails to account for the possibility of multiple initrd paths: if linux-boot-prober gives it multiple paths, separated by spaces, 30_os-prober will remove the spaces and replace them with carets ("^").

The whole system is a hack, but a simple fix is to replace the carets with spaces again, as it does for some of the other fields:

http://git.savannah.gnu.org/cgit/grub.git/tree/util/grub.d/30_os-prober.in
line 210:
- LINITRD="`echo ${LINUX} | cut -d ':' -f 5`"
+ LINITRD="`echo ${LINUX} | cut -d ':' -f 5 | tr '^' ' '`"

NB: linux-boot-prober currently outputs only the first path on the initrd line, even when multiple paths are present.  This bug is being reported separately to the maintainers of os-prober (debian-boot).  The simple fix suggested above assumes that their solution to this bug will be to space-separate multiple paths.

General Chaos <general_chaos>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by nsane457 (Posted a comment)
  • -email is unavailable- added by tpf (Voted in favor of this item)
  • -email is unavailable- added by arvidjaar (Posted a comment)
  • -email is unavailable- added by general_chaos (Submitted the item)
  •  

    There is 1 vote so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-07-01 tpf Carbon-Copy- Added tpf
    2016-04-13 arvidjaar CategoryNone Configuration

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code