GNU GRUB - Bugs: bug #51153, SYSLINUX config parsing is very...
You are not allowed to post comments on this tracker with your current authentication level.
bug #51153: SYSLINUX config parsing is very fragile
Submitter: | felix <felix_s> | ||
Submitted: | Thu 01 Jun 2017 10:00:20 AM UTC | ||
Category: | None | Severity: | Major |
Priority: | 5 - Normal | Item Group: | Software Error |
Status: | None | Privacy: | Public |
Assigned to: | None | Originator Name: | |
Open/Closed: | Open | Release: | Git master |
Release: | Reproducibility: | Every Time | |
Planned Release: | None |
Attached Files
Depends on the following items: None found
Items that depend on this one: None found
Carbon-Copy List
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 |
---|---|---|---|---|---|
2017-06-01 | felix_s | Attached File | - | Added crash.syslinux.cfg, #40831 | |
Attached File | - | Added syslinux.cfg, #40832 |
Powered by Savane 3.13-3230.
Corresponding source code
Attached are two config files. One of them produces a crash (null pointer dereference) when fed to grub-syslinux2cfg or GRUB's syslinux_configfile command; the other produces an unhelpful 'kernel without label' message, without even so much as a line number. Both are understood by vanilla SYSLINUX normally; they are variously processed versons of SYSLINUX config file from SystemRescueCd.
The former bug is because the command-line arguments to ifcpu64.c32 are misparsed (GRUB's parser doesn't understand command-line arguments for kernel alternatives); the latter is because cmd_menudefault doesn't understand a label specifier in a MENU DEFAULT statement.
The SYSLINUX config parsing code generally seems quite fragile. From a very cursory inspection, I've noticed a potential memory leak when encountering multiple DEFAULT statements (cmd_default). A more thorough audit may reveal more such issues.