bugGNU GRUB - Bugs: bug #30362, Unintuitively restrictive grub.cfg...

 
 

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

bug #30362: Unintuitively restrictive grub.cfg parsing (bracketing issue)

Submitter:  Andreas Mohr <andreasmohr>
Submitted:  Mon 05 Jul 2010 10:30:23 PM UTC
   
 
Category:  Configuration Severity:  Major
Priority:  5 - Normal Item Group:  Software Error
Status:  Fixed Privacy:  Public
Assigned to:  None Originator Name: 
Open/Closed:  Closed Release:  1.98
Release:  Reproducibility:  Every Time
Planned Release:  None

Tue 06 Jul 2010 09:22:55 AM UTC, comment #1: 

Hi Andreas,


This issue has been fixed, please try with 1.98+20100705-1 from unstable or grub2 from bzr trunk.


bvk.chaitanya

BVK Chaitanya <bvk>
Group Member
Mon 05 Jul 2010 10:30:23 PM UTC, original submission:  

Hello,

on a grub2 1.98-1 (provided by Debian unstable, installed to a USB stick), I've been wondering for more than half an hour why some friggin' menuentry kept throwing errors ("invalid command", "syntax error" etc. which got displayed in very brief unhelpful flashes during bootup) and consequently did not end up in the menu.

Turned out that the entire menuentry can be nuked by simply adding one space too many after the closing bracket (i.e. one additional whitespace before the newline char).

Sample (as pasted verbatim from http://sidux.com/index.php?name=PNphpBB2&file=viewtopic&t=20371 , ending up WITH problematic bogus whitespace at end despite NOT having it on the web page, at least when using Swiftfox 3.6.3):

menuentry "sidux Moros 2009-04 KDE-lite" {
 loopback loop /boot/iso/sidux-2009-04-moros-kde-lite-i386-200912310312.iso
 linux (loop)/boot/vmlinuz0.686 fromiso=/boot/iso/sidux-2009-04-moros-kde-lite-i386-200912310312.iso boot=fll noeject quiet debug=fll lang=de
 initrd (loop)/boot/initrd0.686
}

This is NOT very helpful, since this means that any menuentry example pasted from internet sources (web browser content) might easily end up with this extra space due to mangling pasting.

As said above, this tiny syntax error took me ages to figure out.
IMHO grub2's parsing should be fixed to be less restrictive in such cases.

Thank you for an otherwise very promising (if still somewhat greenhornish ;) bootloader!

Andreas Mohr <andreasmohr>

 

(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 phcoder (Updated the item)
  • -email is unavailable- added by bvk (Posted a comment)
  • -email is unavailable- added by andreasmohr (Submitted the item)
  •  

    There are 0 votes 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
    2010-08-27 phcoder StatusNone Fixed
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code