83 lines
2.8 KiB
Diff
83 lines
2.8 KiB
Diff
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--)
|