You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
grub2/SOURCES/0211-blscfg-Don-t-attempt-t...

83 lines
2.8 KiB

From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: Javier Martinez Canillas <javierm@redhat.com>
Date: Wed, 18 Jul 2018 00:58:44 +0200
Subject: [PATCH] blscfg: Don't attempt to sort by version if not present in
all BLS files
Commit a16805341cc ("blscfg: sort BLS entries by 'version' field") made to
sort by the version field take precedence over the BLS fragment file name.
But it also uses the lack of the version field in one BLS fragment as sort
criterion, which means that entries could be wrongly sorted if one of them
doesn't have a version field and others do.
So only sort by version if all the BLS entries have this field defined,
otherwise just fallback to sorting by the BLS file name.
Reported-by: Hans de Goede <hdegoede@redhat.com>
Suggested-by: Will Thompson <wjt@endlessm.com>
Signed-off-by: Javier Martinez Canillas <javierm@redhat.com>
---
grub-core/commands/blscfg.c | 27 ++++++++++++++++-----------
1 file changed, 16 insertions(+), 11 deletions(-)
diff --git a/grub-core/commands/blscfg.c b/grub-core/commands/blscfg.c
index 321c93069..69bfb5db2 100644
--- a/grub-core/commands/blscfg.c
+++ b/grub-core/commands/blscfg.c
@@ -324,23 +324,21 @@ finish:
/* return 1: p0 is newer than p1 */
/* 0: p0 and p1 are the same version */
/* -1: p1 is newer than p0 */
-static int bls_cmp(const void *p0, const void *p1, void *state UNUSED)
+static int bls_cmp(const void *p0, const void *p1, void *state)
{
struct bls_entry * e0 = *(struct bls_entry **)p0;
struct bls_entry * e1 = *(struct bls_entry **)p1;
+ bool use_version = *(bool *)state;
const char *v0, *v1;
int r;
- v0 = bls_get_val(e0, "version", NULL);
- v1 = bls_get_val(e1, "version", NULL);
+ if (use_version) {
+ v0 = bls_get_val(e0, "version", NULL);
+ v1 = bls_get_val(e1, "version", NULL);
- if (v0 && !v1)
- return -1;
- if (!v0 && v1)
- return 1;
-
- if ((r = vercmp(v0, v1)) != 0)
- return r;
+ if ((r = vercmp(v0, v1)) != 0)
+ return r;
+ }
return vercmp(e0->filename, e1->filename);
}
@@ -692,6 +690,7 @@ static int find_entry (const char *filename,
grub_device_t blsdir_dev = NULL;
const char *blsdir = NULL;
char *saved_env_buf = NULL;
+ bool use_version = true;
int fallback = 0;
int r = 0;
const char *devid = grub_env_get ("boot");
@@ -819,7 +818,13 @@ read_fallback:
}
grub_dprintf ("blscfg", "Sorting %d entries\n", nentries);
- grub_qsort(&entries[0], nentries, sizeof (struct bls_entry *), bls_cmp, NULL);
+
+ for (r = 0; r < nentries && use_version; r++) {
+ if (!bls_get_val(entries[r], "version", NULL))
+ use_version = false;
+ }
+
+ grub_qsort(&entries[0], nentries, sizeof (struct bls_entry *), bls_cmp, &use_version);
grub_dprintf ("blscfg", "%s Creating %d entries from bls\n", __func__, nentries);
for (r = nentries - 1; r >= 0; r--)