bugGNU GRUB - Bugs: bug #52236, update-grub processing of...

 
 

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

bug #52236: update-grub processing of lilo.conf append line contains a space at the end

Submitter:  Matthew Melton <hairy_marmite>
Submitted:  Mon 16 Oct 2017 02:44:55 PM UTC
   
 
Category:  Configuration Severity:  Major
Priority:  5 - Normal Item Group:  None
Status:  None Privacy:  Public
Assigned to:  None Originator Name:  Matt Melton
Open/Closed:  Open Release:  2.02
Release:  Reproducibility:  None
Planned Release:  None

Mon 16 Oct 2017 02:44:55 PM UTC, original submission:  

This tripped me up during an update (on ubuntu 17.04  grub2/zesty-updates 2.02~beta3-4ubuntu2.2 amd64).

My lilo.conf on another disk caused the upgrade-grub part of the upgrade to fail.
After some quick research documented here:
https://answers.launchpad.net/ubuntu/+question/658067
I found the reason, the following part of my lilo.conf:


image = /boot/vmlinuz
  append="root=PARTUUID=345215fe-01"
  label = Linux
  read-only

Contains a space after the last quote in the append line.
Lilo itself does not worry about this and boots properly.
The dequote function in the bash script "/usr/lib/linux-boot-probes/mounted/50lilo" that parses the lilo.conf file only removes the second quote on the append line if it is the last character on the line.

So essentially a non-critical syntax error in lilo.conf,
caused the update of grub to fail (this was during a kernel security upgrade).

If you need any further details, let me know and I'll do my best to provide them.

Possible fix would be to trim any extra spaces at the end of the line before (or during ) the call to dequote.

All the best

M






Matthew Melton <hairy_marmite>

 

(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 hairy_marmite (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.

     

    No changes have been made to this item

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code