modutils: explain why we no longer quote params
authorDenys Vlasenko <dvlasenk@redhat.com>
Wed, 25 Aug 2010 16:26:46 +0000 (18:26 +0200)
committerDenys Vlasenko <dvlasenk@redhat.com>
Wed, 25 Aug 2010 16:26:46 +0000 (18:26 +0200)
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
modutils/modutils.c

index cc718dbcad6ba46e2561d6dea265937f86b6ffbb..565d0d22f3f852c8c8ae681ec6de81e2e51e1a5d 100644 (file)
@@ -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;