[bug #21865] grub.cfg syntax error leads to infinite loop

[Bean]

Follow-up Comment #3, bug #21865 (project grub):

I write a patch for parser related bug, it fix the following problems:

1, major modification for parse.y, now it should work better. For example,

if echo aa; echo bb; then echo cc; fi

it works properly now.

2, Check for undefined variable. for example, if AA is not defined,

echo $AA

caused problem in the old version. Now it shows blank line.

3, This following commands:

function aa { echo bb; }
aa

Shows

bb
error: unknown command `aa'

Now it is ok.

4, If an command return error in grub.cfg, the following content is not
parsed, for example

test A=B
menuentry aa {
 set root=(hd0,1)
 chainloader +1
}

The menu is not parsed as test A=B set errno to 1.

This bug can be quite tricky, for example, i remember someone report that
update-grub has problem as the menu is not showed at all, but in fact, the
problem is caused by the font command.

5, echo module not included in command.lst
yes, this is problem is very old, but nobody seems to be fixing it.


(file #14684)
    _______________________________________________________

Additional Item Attachment:

File name: grub2-parser-new.diff          Size:6 KB


    _______________________________________________________

Reply to this item at:

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

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