- add \ as quoting character in kernel parameters - generalize quoting character removal to be able to remove quoting character at any position in kernel parameter rather than start and end only This allows passing quotes in kernel arguments. It is also useful to have quoting grammar more similar to shells and bootloaders. Signed-off-by: Michal Suchanek <msuchanek@xxxxxxx> --- - more verbose commit message - optimize final quote removal --- lib/cmdline.c | 50 ++++++++++++++++++++++++++------------------------ 1 file changed, 26 insertions(+), 24 deletions(-) diff --git a/lib/cmdline.c b/lib/cmdline.c index 192848d880ea..aa0086dbb082 100644 --- a/lib/cmdline.c +++ b/lib/cmdline.c @@ -191,32 +191,43 @@ bool parse_option_str(const char *str, const char *option) return false; } +#define break_arg_end(i) { \ + if (isspace(args[i]) && !in_quote && !backslash) \ + break; \ + } + /* * Parse a string to get a param value pair. - * You can use " around spaces, but can't escape ". + * You can use " around spaces, and you can escape with \ * Hyphens and underscores equivalent in parameter names. */ char *next_arg(char *args, char **param, char **val, int *state_flags) { unsigned int i, equals = 0; - int in_quote = 0, quoted = 0; + int in_quote = 0, backslash = 0; char *next; - if (*args == '"') { - args++; - in_quote = 1; - quoted = 1; - } - for (i = 0; args[i]; i++) { - if (isspace(args[i]) && !in_quote) - break; - if (equals == 0) { - if (args[i] == '=') - equals = i; + break_arg_end(i); + + if ((equals == 0) && (args[i] == '=')) + equals = i; + + if (!backslash) { + if ((args[i] == '"') || (args[i] == '\\')) { + if (args[i] == '"') + in_quote = !in_quote; + if (args[i] == '\\') + backslash = 1; + + break_arg_end(i + 1); + memmove(args + 1, args, i); + args++; + i--; + } + } else { + backslash = 0; } - if (args[i] == '"') - in_quote = !in_quote; } if (state_flags) @@ -231,16 +242,7 @@ char *next_arg(char *args, char **param, char **val, int *state_flags) else { args[equals] = '\0'; *val = args + equals + 1; - - /* Don't include quotes in value. */ - if (**val == '"') { - (*val)++; - if (args[i-1] == '"') - args[i-1] = '\0'; - } } - if (quoted && args[i-1] == '"') - args[i-1] = '\0'; if (args[i]) { args[i] = '\0'; -- 2.13.6 -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html