ktest: Fail when grub menu not found
authorSteven Rostedt <srostedt@redhat.com>
Wed, 14 Sep 2011 21:20:39 +0000 (17:20 -0400)
committerSteven Rostedt <rostedt@goodmis.org>
Mon, 17 Oct 2011 15:54:10 +0000 (11:54 -0400)
Currently if the grub menu that is supplied is not found, it will
just boot into the last grub menu in menu.lst. Fail instead of
confusing the user why their kernel is not booting.

Signed-off-by: Steven Rostedt <rostedt@goodmis.org>

tools/testing/ktest/ktest.pl

index debc689..ca6ff99 100755 (executable)
@@ -862,9 +862,12 @@ sub get_grub_index {
     open(IN, "$ssh_grub |")
        or die "unable to get menu.lst";
 
+    my $found = 0;
+
     while (<IN>) {
        if (/^\s*title\s+$grub_menu\s*$/) {
            $grub_number++;
+           $found = 1;
            last;
        } elsif (/^\s*title\s/) {
            $grub_number++;
@@ -873,7 +876,7 @@ sub get_grub_index {
     close(IN);
 
     die "Could not find '$grub_menu' in /boot/grub/menu on $machine"
-       if ($grub_number < 0);
+       if (!$found);
     doprint "$grub_number\n";
 }