From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001 From: Javier Martinez Canillas 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 Suggested-by: Will Thompson Signed-off-by: Javier Martinez Canillas --- 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 dda1bfcc658..8a61ac84dcf 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--)