[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug #63481] 30_os-prober test GRUB_OS_PROBER_SKIP_LIST wrong

From: David Garfield
Subject: [bug #63481] 30_os-prober test GRUB_OS_PROBER_SKIP_LIST wrong
Date: Wed, 7 Dec 2022 06:04:04 -0500 (EST)


                 Summary: 30_os-prober test GRUB_OS_PROBER_SKIP_LIST wrong
                 Project: GNU GRUB
               Submitter: divad27182
               Submitted: Wed 07 Dec 2022 11:04:02 AM UTC
                Category: Configuration
                Severity: Major
                Priority: 5 - Normal
              Item Group: Software Error
                  Status: None
                 Privacy: Public
             Assigned to: None
         Originator Name: David Garfield
        Originator Email: david@barlinq.com
             Open/Closed: Open
                 Release: other
         Discussion Lock: Any
         Reproducibility: Every Time
         Planned Release: None


Follow-up Comments:

Date: Wed 07 Dec 2022 11:04:02 AM UTC By: David Garfield <divad27182>
According to the documentation, the @name clause is added to the test string
for EFI chainloaders.  This is based on os-prober outputting the same clause. 

os-prober output data seems to make the first field be either a device name,
or a device name "@" chainloader path.  Unfortunately, the test for the
presence of the @ reads:

    if [ x"${DEVICE#*@}" != x ] ; then

This tests for the presence of something after the "@", but is always true if
there is no "@".  

I think the test should read:

    if [ x"${DEVICE#*@}" != x"${DEVICE}" ] ; then

It may be that a combined test (differentiating "A", "A@", and "A@B") may be
needed, if os-prober has changed its output format.

This is line 124 of util/grub.d/30_os-prober.in in grub-2.06


Reply to this item at:


Message sent via Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]