dcavalca / rpms / grub2

Forked from rpms/grub2 3 years ago
Clone

Blame SOURCES/0001-Allow-fallback-to-include-entries-by-title-not-just-.patch

0dc71c
From fed4e69879282065708ac7cce9e048f0bb90d51c Mon Sep 17 00:00:00 2001
0dc71c
From: Peter Jones <pjones@redhat.com>
0dc71c
Date: Fri, 5 Sep 2014 10:07:04 -0400
0dc71c
Subject: [PATCH] Allow "fallback" to include entries by title, not just
0dc71c
 number.
0dc71c
0dc71c
Resolves: rhbz#1026084
0dc71c
0dc71c
Signed-off-by: Peter Jones <pjones@redhat.com>
0dc71c
---
0dc71c
 grub-core/normal/menu.c | 76 +++++++++++++++++++++++++++++++------------------
0dc71c
 1 file changed, 49 insertions(+), 27 deletions(-)
0dc71c
0dc71c
diff --git a/grub-core/normal/menu.c b/grub-core/normal/menu.c
0dc71c
index cc27c37..a446d9a 100644
0dc71c
--- a/grub-core/normal/menu.c
0dc71c
+++ b/grub-core/normal/menu.c
0dc71c
@@ -163,12 +163,35 @@ grub_menu_set_timeout (int timeout)
0dc71c
     }
0dc71c
 }
0dc71c
 
0dc71c
+static int
0dc71c
+menuentry_eq (const char *id, const char *spec)
0dc71c
+{
0dc71c
+  const char *ptr1, *ptr2;
0dc71c
+  ptr1 = id;
0dc71c
+  ptr2 = spec;
0dc71c
+  while (1)
0dc71c
+    {
0dc71c
+      if (*ptr2 == '>' && ptr2[1] != '>' && *ptr1 == 0)
0dc71c
+	return 1;
0dc71c
+      if (*ptr2 == '>' && ptr2[1] != '>')
0dc71c
+	return 0;
0dc71c
+      if (*ptr2 == '>')
0dc71c
+	ptr2++;
0dc71c
+      if (*ptr1 != *ptr2)
0dc71c
+	return 0;
0dc71c
+      if (*ptr1 == 0)
0dc71c
+	return 1;
0dc71c
+      ptr1++;
0dc71c
+      ptr2++;
0dc71c
+    }
0dc71c
+}
0dc71c
+
0dc71c
 /* Get the first entry number from the value of the environment variable NAME,
0dc71c
    which is a space-separated list of non-negative integers.  The entry number
0dc71c
    which is returned is stripped from the value of NAME.  If no entry number
0dc71c
    can be found, -1 is returned.  */
0dc71c
 static int
0dc71c
-get_and_remove_first_entry_number (const char *name)
0dc71c
+get_and_remove_first_entry_number (grub_menu_t menu, const char *name)
0dc71c
 {
0dc71c
   const char *val;
0dc71c
   char *tail;
0dc71c
@@ -182,9 +205,32 @@ get_and_remove_first_entry_number (const char *name)
0dc71c
 
0dc71c
   entry = (int) grub_strtoul (val, &tail, 0);
0dc71c
 
0dc71c
+  if (grub_errno == GRUB_ERR_BAD_NUMBER)
0dc71c
+    {
0dc71c
+      /* See if the variable matches the title of a menu entry.  */
0dc71c
+      grub_menu_entry_t e = menu->entry_list;
0dc71c
+      int i;
0dc71c
+
0dc71c
+      grub_errno = GRUB_ERR_NONE;
0dc71c
+
0dc71c
+      for (i = 0; e; i++)
0dc71c
+	{
0dc71c
+	  if (menuentry_eq (e->title, val)
0dc71c
+	      || menuentry_eq (e->id, val))
0dc71c
+	    {
0dc71c
+	      entry = i;
0dc71c
+	      break;
0dc71c
+	    }
0dc71c
+	  e = e->next;
0dc71c
+	}
0dc71c
+
0dc71c
+      if (! e)
0dc71c
+	entry = -1;
0dc71c
+    }
0dc71c
+
0dc71c
   if (grub_errno == GRUB_ERR_NONE)
0dc71c
     {
0dc71c
-      /* Skip whitespace to find the next digit.  */
0dc71c
+      /* Skip whitespace to find the next entry.  */
0dc71c
       while (*tail && grub_isspace (*tail))
0dc71c
 	tail++;
0dc71c
       grub_env_set (name, tail);
0dc71c
@@ -347,7 +393,7 @@ grub_menu_execute_with_fallback (grub_menu_t menu,
0dc71c
   grub_menu_execute_entry (entry, 1);
0dc71c
 
0dc71c
   /* Deal with fallback entries.  */
0dc71c
-  while ((fallback_entry = get_and_remove_first_entry_number ("fallback"))
0dc71c
+  while ((fallback_entry = get_and_remove_first_entry_number (menu, "fallback"))
0dc71c
 	 >= 0)
0dc71c
     {
0dc71c
       grub_print_error ();
0dc71c
@@ -465,30 +511,6 @@ grub_menu_register_viewer (struct grub_menu_viewer *viewer)
0dc71c
   viewers = viewer;
0dc71c
 }
0dc71c
 
0dc71c
-static int
0dc71c
-menuentry_eq (const char *id, const char *spec)
0dc71c
-{
0dc71c
-  const char *ptr1, *ptr2;
0dc71c
-  ptr1 = id;
0dc71c
-  ptr2 = spec;
0dc71c
-  while (1)
0dc71c
-    {
0dc71c
-      if (*ptr2 == '>' && ptr2[1] != '>' && *ptr1 == 0)
0dc71c
-	return 1;
0dc71c
-      if (*ptr2 == '>' && ptr2[1] != '>')
0dc71c
-	return 0;
0dc71c
-      if (*ptr2 == '>')
0dc71c
-	ptr2++;
0dc71c
-      if (*ptr1 != *ptr2)
0dc71c
-	return 0;
0dc71c
-      if (*ptr1 == 0)
0dc71c
-	return 1;
0dc71c
-      ptr1++;
0dc71c
-      ptr2++;
0dc71c
-    }
0dc71c
-}
0dc71c
-
0dc71c
-
0dc71c
 /* Get the entry number from the variable NAME.  */
0dc71c
 static int
0dc71c
 get_entry_number (grub_menu_t menu, const char *name)
0dc71c
-- 
0dc71c
1.9.3
0dc71c