Implement multiboot filename duplication in legacy parser

diff --git a/grub-core/commands/legacycfg.c b/grub-core/commands/legacycfg.c
index aca6d1e1f951f5da8c28e5b8330d3e07a77e681b..100464e69e1dd364e74490167ed61b9c0f789205 100644 (file)
--- a/grub-core/commands/legacycfg.c
+++ b/grub-core/commands/legacycfg.c
@@ -179,6 +179,8 @@ grub_cmd_legacy_kernel (struct grub_command *mycmd __attribute__ ((unused)),
   int i;
   int no_mem_option = 0;
   struct grub_command *cmd;
+  char **cutargs;
+  int cutargc;
   for (i = 0; i < 2; i++)
     {
       /* FIXME: really support this.  */
@@ -233,9 +235,14 @@ grub_cmd_legacy_kernel (struct grub_command *mycmd __attribute__ ((unused)),
        }
     }
 
-  if (!argc)
+  if (argc < 2)
     return grub_error (GRUB_ERR_BAD_ARGUMENT, "filename required");
 
+  cutargs = grub_malloc (sizeof (cutargsp[0]) * (argc - 1));
+  cutargc = argc - 1;
+  grub_memcpy (cutargs + 1, args + 2, sizeof (cutargsp[0]) * (argc - 2));
+  cutargs[0] = args[0];
+
   do
     {
       /* First try Linux.  */
@@ -244,7 +251,7 @@ grub_cmd_legacy_kernel (struct grub_command *mycmd __attribute__ ((unused)),
          cmd = grub_command_find ("linux16");
          if (cmd)
            {
-             if (!(cmd->func) (cmd, argc, args))
+             if (!(cmd->func) (cmd, cutargc, cutargs))
                {
                  kernel_type = LINUX;
                  return GRUB_ERR_NONE;
@@ -275,7 +282,7 @@ grub_cmd_legacy_kernel (struct grub_command *mycmd __attribute__ ((unused)),
          cmd = grub_command_find ("kfreebsd");
          if (cmd)
            {
-             if (!(cmd->func) (cmd, argc, args))
+             if (!(cmd->func) (cmd, cutargc, cutargs))
                {
                  kernel_type = KFREEBSD;
                  return GRUB_ERR_NONE;
@@ -288,7 +295,7 @@ grub_cmd_legacy_kernel (struct grub_command *mycmd __attribute__ ((unused)),
          cmd = grub_command_find ("knetbsd");
          if (cmd)
            {
-             if (!(cmd->func) (cmd, argc, args))
+             if (!(cmd->func) (cmd, cutargc, cutargs))
                {
                  kernel_type = KNETBSD;
                  return GRUB_ERR_NONE;
@@ -301,7 +308,7 @@ grub_cmd_legacy_kernel (struct grub_command *mycmd __attribute__ ((unused)),
          cmd = grub_command_find ("kopenbsd");
          if (cmd)
            {
-             if (!(cmd->func) (cmd, argc, args))
+             if (!(cmd->func) (cmd, cutargc, cutargs))
                {
                  kernel_type = KOPENBSD;
                  return GRUB_ERR_NONE;

diff --git a/grub-core/lib/legacy_parse.c b/grub-core/lib/legacy_parse.c
index f350aaf0980be5dc220e0db7f840c3e57ffd0b22..694de097bc063ad4b4ef353ba992ba047f63a9c7 100644 (file)
--- a/grub-core/lib/legacy_parse.c
+++ b/grub-core/lib/legacy_parse.c
@@ -33,6 +33,7 @@ struct legacy_command
     TYPE_NOAPM_OPTION,
     TYPE_TYPE_OR_NOMEM_OPTION,
     TYPE_FILE,
+    TYPE_FILE_NO_CONSUME,
     TYPE_PARTITION,
     TYPE_BOOL,
     TYPE_INT,
@@ -113,7 +114,8 @@ struct legacy_command legacy_commands[] =
     /* ifconfig unsupported.  */
     /* impsprobe unsupported.  */
     /* FIXME: dublicate multiboot filename. */
-    {"initrd", "legacy_initrd '%s' %s\n", 2, {TYPE_FILE, TYPE_REST_VERBATIM}, 0,
+    {"initrd", "legacy_initrd '%s' %s\n", 2, {TYPE_FILE_NO_CONSUME,
+                                             TYPE_REST_VERBATIM}, 0,
      "FILE [ARG ...]",
      "Load an initial ramdisk FILE for a Linux format boot image and set the"
      " appropriate parameters in the Linux setup area in memory."},
@@ -122,9 +124,9 @@ struct legacy_command legacy_commands[] =
     /* FIXME: really support --no-mem-option.  */
     /* FIXME: dublicate multiboot filename. */
     {"kernel", "legacy_kernel %s %s '%s' %s\n", 4, {TYPE_TYPE_OR_NOMEM_OPTION,
-                                                TYPE_TYPE_OR_NOMEM_OPTION,
-                                                TYPE_FILE,
-                                                TYPE_REST_VERBATIM}, 0,
+                                                   TYPE_TYPE_OR_NOMEM_OPTION,
+                                                   TYPE_FILE_NO_CONSUME,
+                                                   TYPE_REST_VERBATIM}, 0,
      "[--no-mem-option] [--type=TYPE] FILE [ARG ...]",
      "Attempt to load the primary boot image from FILE. The rest of the"
      " line is passed verbatim as the \"kernel command line\".  Any modules"
@@ -143,7 +145,8 @@ struct legacy_command legacy_commands[] =
      " when you chain-load some operating systems, such as DOS, if such an"
      " OS resides at a non-first drive."},
     /* md5crypt unsupported.  */
-    {"module", "legacy_initrd '%s' %s\n", 1, {TYPE_FILE, TYPE_REST_VERBATIM}, 0,
+    {"module", "legacy_initrd '%s' %s\n", 1, {TYPE_FILE_NO_CONSUME,
+                                             TYPE_REST_VERBATIM}, 0,
      "FILE [ARG ...]",
      "Load a boot module FILE for a Multiboot format boot image (no"
      " interpretation of the file contents is made, so users of this"
@@ -210,7 +213,6 @@ struct legacy_command legacy_commands[] =
      " compares them, to test the filesystem code. "
      " If this test succeeds, then a good next"
      " step is to try loading a kernel."},
-     "Print the contents of the file FILE."},
     /* testvbe unsupported.  */
     /* tftpserver unsupported.  */
     {"timeout", "set timeout=%s\n", 1, {TYPE_INT}, 0, "SEC",
@@ -399,6 +401,8 @@ grub_legacy_parse (const char *buf, char **entryname)
          ptr++;
        switch (legacy_commands[cmdnum].argt[i])
          {
+         case TYPE_FILE_NO_CONSUME:
+           hold_arg = 1;
          case TYPE_PARTITION:
          case TYPE_FILE:
            args[j++] = adjust_file (curarg, curarglen);