modutils: explain why we no longer quote params

Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>

1 files changed, 5 insertions, 0 deletions

diff --git a/modutils/modutils.c b/modutils/modutils.c
index cc718dbca..565d0d22f 100644
--- a/modutils/modutils.c
+++ b/modutils/modutils.c
@@ -71,6 +71,11 @@ char* FAST_FUNC parse_cmdline_module_options(char **argv)
         optlen = 0;
         while (*++argv) {
                 options = xrealloc(options, optlen + 2 + strlen(*argv) + 2);
+                /* Older versions were enclosing space-containing *argv in "",
+                 * but both modprobe and insmod from module-init-tools 3.11.1
+                 * don't do this anymore. (As to extra trailing space,
+                 * insmod adds it but modprobe does not. We do in both cases)
+                 */
                 optlen += sprintf(options + optlen, "%s ", *argv);
         }
         return options;