[git commit master] sed: fix a command with multible trailing backslashes

Denys Vlasenko vda.linux at googlemail.com
Tue May 11 23:49:04 UTC 2010


commit: http://git.busybox.net/busybox/commit/?id=a2215b98f7d65bc613b9c8f008d79672402c6a07
branch: http://git.busybox.net/busybox/commit/?id=refs/heads/master

Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>
---
 editors/sed.c        |   14 ++++++++------
 testsuite/sed.tests  |   14 ++++++++++++++
 testsuite/testing.sh |    2 +-
 3 files changed, 23 insertions(+), 7 deletions(-)
 mode change 100755 => 100644 testsuite/testing.sh

diff --git a/editors/sed.c b/editors/sed.c
index 30ab8c9..4bd6e01 100644
--- a/editors/sed.c
+++ b/editors/sed.c
@@ -487,7 +487,7 @@ static const char *parse_cmd_args(sed_cmd_t *sed_cmd, const char *cmdstr)
 static void add_cmd(const char *cmdstr)
 {
 	sed_cmd_t *sed_cmd;
-	int temp;
+	unsigned len, n;
 
 	/* Append this line to any unfinished line from last time. */
 	if (G.add_cmd_line) {
@@ -496,12 +496,14 @@ static void add_cmd(const char *cmdstr)
 		cmdstr = G.add_cmd_line = tp;
 	}
 
-	/* If this line ends with backslash, request next line. */
-	temp = strlen(cmdstr);
-	if (temp && cmdstr[--temp] == '\\') {
+	/* If this line ends with unescaped backslash, request next line. */
+	n = len = strlen(cmdstr);
+	while (n && cmdstr[n-1] == '\\')
+		n--;
+	if ((len - n) & 1) { /* if odd number of trailing backslashes */
 		if (!G.add_cmd_line)
 			G.add_cmd_line = xstrdup(cmdstr);
-		G.add_cmd_line[temp] = '\0';
+		G.add_cmd_line[len-1] = '\0';
 		return;
 	}
 
@@ -560,7 +562,7 @@ static void add_cmd(const char *cmdstr)
 		/* last part (mandatory) will be a command */
 		if (!*cmdstr)
 			bb_error_msg_and_die("missing command");
-		sed_cmd->cmd = *(cmdstr++);
+		sed_cmd->cmd = *cmdstr++;
 		cmdstr = parse_cmd_args(sed_cmd, cmdstr);
 
 		/* Add the command to the command array */
diff --git a/testsuite/sed.tests b/testsuite/sed.tests
index b0de965..5b0750c 100755
--- a/testsuite/sed.tests
+++ b/testsuite/sed.tests
@@ -258,4 +258,18 @@ testing "sed nested {}s" \
 	"qwe\nasd\nacd\nacd\n" "" \
 	"qwe\nasd\nzxc\n"
 
+testing "sed a cmd ended by double backslash" \
+	"sed -e '/| one /a \\
+	| three \\\\' -e '/| one-/a \\
+	| three-* \\\\'" \
+'	| one \\
+	| three \\
+	| two \\
+' '' \
+'	| one \\
+	| two \\
+'
+
+# testing "description" "arguments" "result" "infile" "stdin"
+
 exit $FAILCOUNT
diff --git a/testsuite/testing.sh b/testsuite/testing.sh
old mode 100755
new mode 100644
index 65a0f65..913d7f8
--- a/testsuite/testing.sh
+++ b/testsuite/testing.sh
@@ -73,7 +73,7 @@ testing()
 
   if [ $# -ne 5 ]
   then
-    echo "Test $NAME has wrong number of arguments (must be 5) ($# $*)" >&2
+    echo "Test $NAME has wrong number of arguments: $# (must be 5)" >&2
     exit 1
   fi
 
-- 
1.6.3.3



More information about the busybox-cvs mailing list