diff options
author | Oleksii Shevchuk <alxchk@gmail.com> | 2012-11-03 21:52:02 +0200 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2012-11-15 16:01:10 +0100 |
commit | 7e1a84f55244ca78093b1dabc58683bc0e7f4304 (patch) | |
tree | 03eeff2c5547f8188bc3309e704881e836fbcc63 | |
parent | 2c5417ade0d137f811e4fcc1b099f963e6d5a820 (diff) | |
download | systemd-7e1a84f55244ca78093b1dabc58683bc0e7f4304.tar.gz |
core: interpret \; token in ExecStart as escaped ;
Some commands (like 'find') take a semicolon as separate arg. With
current parser implementation there is no way to pass one.
Patch adds token \;
-rw-r--r-- | src/core/load-fragment.c | 2 | ||||
-rw-r--r-- | src/test/test-unit-file.c | 10 |
2 files changed, 12 insertions, 0 deletions
diff --git a/src/core/load-fragment.c b/src/core/load-fragment.c index 5803044178..4dc5c529a2 100644 --- a/src/core/load-fragment.c +++ b/src/core/load-fragment.c @@ -483,6 +483,8 @@ int config_parse_exec( FOREACH_WORD_QUOTED(w, l, rvalue, state) { if (strncmp(w, ";", MAX(l, 1U)) == 0) break; + else if (strncmp(w, "\\;", MAX(l, 1U)) == 0) + w ++; if (honour_argv0 && w == rvalue) { assert(!path); diff --git a/src/test/test-unit-file.c b/src/test/test-unit-file.c index bb5cbdf9d7..bca8a69b78 100644 --- a/src/test/test-unit-file.c +++ b/src/test/test-unit-file.c @@ -138,6 +138,16 @@ static void test_config_parse_exec(void) { assert_se(c1->command_next == NULL); + /* escaped semicolon */ + r = config_parse_exec("fake", 5, "section", + "LValue", 0, + "/usr/bin/find \\;", + &c, NULL); + assert_se(r >= 0); + c1 = c1->command_next; + check_execcommand(c1, + "/usr/bin/find", "/usr/bin/find", ";", false); + exec_command_free_list(c); } |