diff options
| author |   Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2011-11-23 20:53:17 +0100 |
|---|---|---|
| committer |   Nicholas Bellinger <nab@linux-iscsi.org> | 2011-12-14 11:27:23 +0000 |
| commit | 90c161b643d9531d271110876a14e68b49172d8a (patch) | |
| tree | 2427163f641294c15e1fc881993994f0ebe26d8b /drivers/target/target_core_configfs.c | |
| parent | target: make the se_task task_state_active a normal bool (diff) | |
| download | linux-dev-90c161b643d9531d271110876a14e68b49172d8a.tar.xz linux-dev-90c161b643d9531d271110876a14e68b49172d8a.zip | |
target: use \n as a separator for configuration
The command
| echo rd_pages=32768 > ramdisk/control
Does not work because it writes "rd_pages=32768\n" and the parser which
matches for "rd_pages=%d" does not recognize it due to the \n. One way
of fixing this would be using "echo -n" instead.
This patch adds \n to the list of separators so we don't have to use the
-n argument which I find is more convinient.
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'drivers/target/target_core_configfs.c')
| -rw-r--r-- | drivers/target/target_core_configfs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/target/target_core_configfs.c b/drivers/target/target_core_configfs.c index 3c358907a4a6..37b2b3d03c65 100644 --- a/drivers/target/target_core_configfs.c +++ b/drivers/target/target_core_configfs.c @@ -1450,7 +1450,7 @@ static ssize_t target_core_dev_pr_store_attr_res_aptpl_metadata( return -ENOMEM; orig = opts; - while ((ptr = strsep(&opts, ",")) != NULL) { + while ((ptr = strsep(&opts, ",\n")) != NULL) { if (!*ptr) continue; |