diff options
author | James Minor <james.minor@ni.com> | 2017-01-25 18:13:32 -0600 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2017-01-31 14:43:01 +0000 |
commit | 1f7c9094f703c234c9bef6090b958dbe28ec7374 (patch) | |
tree | 55a52db3174bc8f560a6bf50d632ddaf22921014 | |
parent | 50c885e165ac095332cacd010476d013567c4040 (diff) | |
download | poky-1f7c9094f703c234c9bef6090b958dbe28ec7374.tar.gz |
kmod: Fix handling of quotes in kernel command line
If a module parameter on the command line contains quotes, any
spaces inside those quotes should be included as part of the
parameter.
Upstream-Status: Accepted
(From OE-Core rev: a54be23887cdc24a9b128be0913029fc8e63973d)
Signed-off-by: James Minor <james.minor@ni.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | meta/recipes-kernel/kmod/kmod.inc | 1 | ||||
-rw-r--r-- | meta/recipes-kernel/kmod/kmod/kcmdline_quotes.patch | 44 |
2 files changed, 45 insertions, 0 deletions
diff --git a/meta/recipes-kernel/kmod/kmod.inc b/meta/recipes-kernel/kmod/kmod.inc index a59c30586e..cfa409dbb3 100644 --- a/meta/recipes-kernel/kmod/kmod.inc +++ b/meta/recipes-kernel/kmod/kmod.inc | |||
@@ -22,6 +22,7 @@ SRC_URI = "git://git.kernel.org/pub/scm/utils/kernel/kmod/kmod.git \ | |||
22 | file://depmod-search.conf \ | 22 | file://depmod-search.conf \ |
23 | file://avoid_parallel_tests.patch \ | 23 | file://avoid_parallel_tests.patch \ |
24 | file://fix-O_CLOEXEC.patch \ | 24 | file://fix-O_CLOEXEC.patch \ |
25 | file://kcmdline_quotes.patch \ | ||
25 | " | 26 | " |
26 | 27 | ||
27 | S = "${WORKDIR}/git" | 28 | S = "${WORKDIR}/git" |
diff --git a/meta/recipes-kernel/kmod/kmod/kcmdline_quotes.patch b/meta/recipes-kernel/kmod/kmod/kcmdline_quotes.patch new file mode 100644 index 0000000000..46bdec59a7 --- /dev/null +++ b/meta/recipes-kernel/kmod/kmod/kcmdline_quotes.patch | |||
@@ -0,0 +1,44 @@ | |||
1 | From 4a6f92a10680e7e36807f5e2ae8e497e8d73a048 Mon Sep 17 00:00:00 2001 | ||
2 | From: James Minor <james.minor@ni.com> | ||
3 | Date: Fri, 20 Jan 2017 17:15:50 -0600 | ||
4 | Subject: [PATCH] libkmod: Fix handling of quotes in kernel command line | ||
5 | |||
6 | If a module parameter on the command line contains quotes, any | ||
7 | spaces inside those quotes should be included as part of the | ||
8 | parameter. | ||
9 | |||
10 | Signed-off-by: James Minor <james.minor@ni.com> | ||
11 | |||
12 | Upstream-Status: Accepted | ||
13 | --- | ||
14 | libkmod/libkmod-config.c | 7 +++++++ | ||
15 | 1 file changed, 7 insertions(+) | ||
16 | |||
17 | diff --git a/libkmod/libkmod-config.c b/libkmod/libkmod-config.c | ||
18 | index 57fbe37..ea40d19 100644 | ||
19 | --- a/libkmod/libkmod-config.c | ||
20 | +++ b/libkmod/libkmod-config.c | ||
21 | @@ -497,6 +497,7 @@ static int kmod_config_parse_kcmdline(struct kmod_config *config) | ||
22 | char buf[KCMD_LINE_SIZE]; | ||
23 | int fd, err; | ||
24 | char *p, *modname, *param = NULL, *value = NULL, is_module = 1; | ||
25 | + bool is_quoted = false; | ||
26 | |||
27 | fd = open("/proc/cmdline", O_RDONLY|O_CLOEXEC); | ||
28 | if (fd < 0) { | ||
29 | @@ -514,6 +515,12 @@ static int kmod_config_parse_kcmdline(struct kmod_config *config) | ||
30 | } | ||
31 | |||
32 | for (p = buf, modname = buf; *p != '\0' && *p != '\n'; p++) { | ||
33 | + if (*p == '"') { | ||
34 | + is_quoted = !is_quoted; | ||
35 | + continue; | ||
36 | + } | ||
37 | + if (is_quoted) | ||
38 | + continue; | ||
39 | switch (*p) { | ||
40 | case ' ': | ||
41 | *p = '\0'; | ||
42 | -- | ||
43 | 1.9.1 | ||
44 | |||