insmod: Do not add a pair of "" around the arguments of the module.
authorCarmelo AMOROSO <carmelo.amoroso@st.com>
Tue, 24 Aug 2010 04:01:13 +0000 (06:01 +0200)
committerDenys Vlasenko <dvlasenk@redhat.com>
Wed, 25 Aug 2010 16:19:22 +0000 (18:19 +0200)
If there are some spaces in the insmod command line, then
this will be splitted in single words as separate elements of
argv. It just needs to chain them together in the options string
passed to the sys_init_module syscall.

Signed-off-by: Carmelo Amoroso <carmelo.amoroso@st.com>
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
modutils/modutils.c

index 2608182a172e330a4dd20f21816a7efaa460c7df..cc718dbcad6ba46e2561d6dea265937f86b6ffbb 100644 (file)
@@ -71,8 +71,7 @@ char* FAST_FUNC parse_cmdline_module_options(char **argv)
        optlen = 0;
        while (*++argv) {
                options = xrealloc(options, optlen + 2 + strlen(*argv) + 2);
-               /* Spaces handled by "" pairs, but no way of escaping quotes */
-               optlen += sprintf(options + optlen, (strchr(*argv, ' ') ? "\"%s\" " : "%s "), *argv);
+               optlen += sprintf(options + optlen, "%s ", *argv);
        }
        return options;
 }